Page MenuHomePhorge
Feed All Stories

Mon, Oct 14

keithzg added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

Many thanks, @20after4! That all largely did it; in addition to setting up the object a bit wrong (I knew it was a ManiphestTask instance, but was failing to get the actual object of it), I was also just using essentially my:custom-field rather than std:maniphest:my:custom-field.

Mon, Oct 14, 23:07
Tgr updated the task description for T15949: Herald comment attributed to the wrong user in email notifications.
Mon, Oct 14, 19:10 · Herald, Maniphest
Tgr created T15949: Herald comment attributed to the wrong user in email notifications.
Mon, Oct 14, 19:09 · Herald, Maniphest
s.quinn closed Q153: php-fpm memory leak as invalid.
Mon, Oct 14, 13:44
s.quinn added a comment to Q153: php-fpm memory leak.

Appears to be internal monitoring software causing this (not phorge), thanks for the replies.

Mon, Oct 14, 13:43
valerio.bozzolan closed Q156: How to support project commercialy? as resolved.
Mon, Oct 14, 13:25
valerio.bozzolan updated the answer details for Q156: How to support project commercialy? (Answer 193).
Mon, Oct 14, 13:24
valerio.bozzolan added a member for Trusted Contributors: syncer.
Mon, Oct 14, 13:13
valerio.bozzolan added Q156: How to support project commercialy? (Answer 193).
Mon, Oct 14, 07:16

Thu, Oct 10

syncer updated the question details for Q156: How to support project commercialy?.
Thu, Oct 10, 23:09
syncer asked Q156: How to support project commercialy?.
Thu, Oct 10, 22:48
sirocyl added a comment to T15630: Fix Fonts.

Noting a user complaint (I'm using a mildly hacked-up Phorge to host https://blog.styx-os.org/ - but the font stack is unchanged): https://fe.disroot.org/objects/5d376f6a-0acc-44fc-bd37-31de8630b647

Thu, Oct 10, 09:21
avivey awarded T15948: Allow public API access when equivalent public web access exists a Like token.
Thu, Oct 10, 06:54 · Conduit, Phorge

Wed, Oct 9

20after4 added a comment to T15947: Add method to query if user is member of a list of projects.

Maybe express it as allowed by Policy xxx? (Related to T15277)

Wed, Oct 9, 18:23 · Feature Requests
20after4 added a comment to T15947: Add method to query if user is member of a list of projects.

So essentially this:

Wed, Oct 9, 18:17 · Feature Requests
20after4 added a comment to T15948: Allow public API access when equivalent public web access exists.

If I were starting today I would probably design the back-end APIs first, then make the web interface derive from those APIs, such that web requests and api requests are not really any different, at least with regard to enforcing access controls.

Wed, Oct 9, 18:07 · Conduit, Phorge
valerio.bozzolan awarded T15948: Allow public API access when equivalent public web access exists a Mountain of Wealth token.
Wed, Oct 9, 14:56 · Conduit, Phorge
fgaz created T15948: Allow public API access when equivalent public web access exists.
Wed, Oct 9, 14:25 · Conduit, Phorge
avivey added a comment to T15947: Add method to query if user is member of a list of projects.

Maybe express it as allowed by Policy xxx? (Related to T15277)

Wed, Oct 9, 10:29 · Feature Requests
aklapper created T15947: Add method to query if user is member of a list of projects.
Wed, Oct 9, 09:46 · Feature Requests
valerio.bozzolan edited the content of 2024 week 35.
Wed, Oct 9, 09:36
aklapper closed T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view as Resolved by committing rPb0bf1c689a9d: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit….
Wed, Oct 9, 09:19
aklapper closed D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view.
Wed, Oct 9, 09:19
aklapper committed rPb0bf1c689a9d: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit….
Avoid RuntimeException passing bogus "service" parameter to Diffusion commit…
Wed, Oct 9, 09:19
aklapper added a comment to T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view.

No clue :)

Wed, Oct 9, 09:17
valerio.bozzolan added a comment to D25827: Allow zooming on pages.

P.S. maybe relevant - maybe not :D lol https://we.phorge.it/T15945#19568

Wed, Oct 9, 08:26
20after4 accepted D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view.
Wed, Oct 9, 08:07
20after4 awarded D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view a Yellow Medal token.
Wed, Oct 9, 08:07
20after4 awarded D25827: Allow zooming on pages a Yellow Medal token.
Wed, Oct 9, 08:06
20after4 accepted D25827: Allow zooming on pages.

This one seems worthy of a rubber stamp. And a gold star.

Wed, Oct 9, 08:05
20after4 updated the answer details for Q155: Is there a way to add new tokens? (Answer 192).
Wed, Oct 9, 08:03
20after4 added Q155: Is there a way to add new tokens? (Answer 192).
Wed, Oct 9, 08:00
20after4 added a comment to Q153: php-fpm memory leak.

Are you importing some huge git repo or something? I haven't seen a lot of OOM with php-fpm on phorge/phabricator. It's generally not been an issue in the past so this is either a new bug or something specific to the environment. Then again, I've almost always ran Phorge with much more than 8GB of ram available.

Wed, Oct 9, 07:50
20after4 added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

For another example, here is a fairly straightforward use of that API which takes place entirely outside of a custom field subclass:

Wed, Oct 9, 07:41
20after4 added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

There is a pretty complicated and not really easy to follow subclass of ManiphestCustomField over here: thttps://phabricator.wikimedia.org/source/phab-extensions/browse/wmf%252Fstable/src/customfields/ReleaseDetailsCustomField.php

Wed, Oct 9, 07:33
20after4 added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

@keithzg what you might be missing is

$object = $this->getObject()
Wed, Oct 9, 07:31
20after4 updated subscribers of T15501: Voluntary Usage Survey App.

There is a significant amount of Phabricator dark matter out there - companies/people using the software, it works well enough, not really easy to know they exist or anything about their usage. I'm sure at least some of them have moved to Phorge. Automattic/wordpress.com have moved to Phorge and I wasn't even aware that they were using Phabricator before that. This is despite the fact that I did a pretty extensive amount of research to identify every company using Phabricator back in ~2019 as part of my work for Wikimedia, with the goal of reaching out and trying to organize an informal Phabricator users group. We had the idea that the various corporate users probably had good reasons to be collaborating and at least talking to each other since most of them were not active in the upstream project. Anyway, that never really panned out, although it did trigger a flurry of interest and some ongoing discussions via email (maybe even one video meeting but I can't remember the details now.)

Wed, Oct 9, 07:16 · Discussion Needed

Tue, Oct 8

fgaz created T15946: Support downloading a compressed working copy (tar/zip) of a repository at a specific ref (commit, branch, or tag).
Tue, Oct 8, 17:14 · Diffusion, Phorge
valerio.bozzolan added a comment to T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view.

Any guess about a supposed right value for service? lol Just for my curiosity

Tue, Oct 8, 12:12

Wed, Oct 2

valerio.bozzolan added a comment to T15945: Allow zooming Phorge pages on mobile.

Just talking about accessibility, maybe interesting, Wikipedia on mobile allows to scale, but setting a minimum and maximum scale:

Wed, Oct 2, 14:47 · Accessibility
aklapper updated the summary of D25827: Allow zooming on pages.
Wed, Oct 2, 12:45
aklapper requested review of D25827: Allow zooming on pages.
Wed, Oct 2, 12:44
aklapper added a revision to T15945: Allow zooming Phorge pages on mobile: D25827: Allow zooming on pages.
Wed, Oct 2, 12:44 · Accessibility
BlankEclair updated the answer wiki for Q155: Is there a way to add new tokens?.
Wed, Oct 2, 11:53
BlankEclair added a comment to Q155: Is there a way to add new tokens? (Answer 191).

Thanks for the very detailed answer! I may or may not take a stab at this if the person who originally asked for it decides that this is worth the effort.

Wed, Oct 2, 11:52
valerio.bozzolan updated the answer details for Q155: Is there a way to add new tokens? (Answer 191).
Wed, Oct 2, 09:54
valerio.bozzolan updated the answer details for Q155: Is there a way to add new tokens? (Answer 191).
Wed, Oct 2, 09:49
valerio.bozzolan added Q155: Is there a way to add new tokens? (Answer 191).
Wed, Oct 2, 09:48
BlankEclair asked Q155: Is there a way to add new tokens?.
Wed, Oct 2, 04:40

Tue, Oct 1

aklapper created T15945: Allow zooming Phorge pages on mobile.
Tue, Oct 1, 18:17 · Accessibility
aklapper added inline comments to D25825: Validate some user provided calendar query range dates.
Tue, Oct 1, 15:19
aklapper added a comment to D25738: Avoid RuntimeException on "Skip past this commit" when commit still importing.

Stumbled upon https://we.phorge.it/rP95e179d9a4f31b8c5c2cbb4db8f7fa9f2d3867d6 which also mentions

We may not have a commit object for a given identifier if the commit has not imported yet.

So this can indeed happen.

Tue, Oct 1, 15:12
aklapper requested review of D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view.
Tue, Oct 1, 15:10
aklapper added a revision to T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view: D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view.
Tue, Oct 1, 15:10
aklapper created T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view.
Tue, Oct 1, 14:35

Mon, Sep 30

valerio.bozzolan added a comment to Q153: php-fpm memory leak.

free -h ?

Mon, Sep 30, 14:30
aklapper closed T15836: ArcanistHardpointTask calls non-existent constructor for ArcanistHardpointFutureList as Resolved by committing rARC285e31f4c743: Fix invalid constructor call for ArcanistHardpointFutureList.
Mon, Sep 30, 13:36 · Arcanist
aklapper closed D25708: Fix invalid constructor call for ArcanistHardpointFutureList.
Mon, Sep 30, 13:36
aklapper committed rARC285e31f4c743: Fix invalid constructor call for ArcanistHardpointFutureList.
Fix invalid constructor call for ArcanistHardpointFutureList
Mon, Sep 30, 13:36
valerio.bozzolan closed Q154: Modifying Celerity postprocessors - is there a cache I should flush? as resolved.
Mon, Sep 30, 11:29
sirocyl added Q154: Modifying Celerity postprocessors - is there a cache I should flush? (Answer 190).
Mon, Sep 30, 09:08
sirocyl added a comment to Q154: Modifying Celerity postprocessors - is there a cache I should flush?.

Ah, hadn't even thought to check Developer Mode, I assumed I had it enabled. I did a "Purge Caches" from there and that didn't do anything, but turning devmode on did.

Mon, Sep 30, 09:06
avivey added a comment to Q154: Modifying Celerity postprocessors - is there a cache I should flush?.

it might be in the APCu cache (https://we.phorge.it/config/cache/) - I'll try to check later. Also make sure you've set the Developer Mode in settings.

Mon, Sep 30, 06:34
sirocyl asked Q154: Modifying Celerity postprocessors - is there a cache I should flush?.
Mon, Sep 30, 01:11

Sat, Sep 28

avivey added a comment to D25823: arc browse: add support to Subversion repos.

Ok, so --branch probably doesn't make much sense in that case, right.

Sat, Sep 28, 18:58

Thu, Sep 26

valerio.bozzolan added a comment to D25823: arc browse: add support to Subversion repos.

mm, isBranchJustFilesystemPath() makes more sense now, as in "the branch name is part of the URI path"...

Thu, Sep 26, 14:54
avivey added a comment to D25823: arc browse: add support to Subversion repos.

mm, isBranchJustFilesystemPath() makes more sense now, as in "the branch name is part of the URI path"...

Thu, Sep 26, 13:59
valerio.bozzolan added a comment to D25823: arc browse: add support to Subversion repos.

Maybe I should just try to improve the ArcanistRepositoryAPI#getDefaultBranch() 🤔 since the root problem is there, and the current solution just involves more code to avoid to fallback on that "master" default

Thu, Sep 26, 09:54

Sep 25 2024

s.quinn added a comment to Q35: Does Phorge provide single sign-on to Microsoft Azure B2C? (Answer 53).

done

Sep 25 2024, 18:29
valerio.bozzolan updated the diff for D25823: arc browse: add support to Subversion repos.

move confusing comment

Sep 25 2024, 15:35
valerio.bozzolan added a comment to D25823: arc browse: add support to Subversion repos.

a. I'm not sure I like isBranchJustFilesystemPath() to check Is this SVN - if we're trying to be generic, then there may be a different vcs that does this particular trick, but has different logic; And if we're not trying to be generic, then isSVN() is probably a good-ish name? or isVCS('SVN').

Sep 25 2024, 15:29
valerio.bozzolan updated the diff for D25823: arc browse: add support to Subversion repos.

wow, it also makes sense to add --branch support to SVN! added

Sep 25 2024, 15:09
valerio.bozzolan updated the test plan for D25823: arc browse: add support to Subversion repos.
Sep 25 2024, 15:08
valerio.bozzolan updated the diff for D25823: arc browse: add support to Subversion repos.

integrate tips

Sep 25 2024, 14:27
avivey added a comment to D25823: arc browse: add support to Subversion repos.

a. I'm not sure I like isBranchJustFilesystemPath() to check Is this SVN - if we're trying to be generic, then there may be a different vcs that does this particular trick, but has different logic; And if we're not trying to be generic, then isSVN() is probably a good-ish name? or isVCS('SVN').

Sep 25 2024, 14:01
valerio.bozzolan added a comment to Q35: Does Phorge provide single sign-on to Microsoft Azure B2C? (Answer 53).

Visit this page: https://we.phorge.it/F2517458 and Edit thanks

Sep 25 2024, 12:57
s.quinn added a comment to Q35: Does Phorge provide single sign-on to Microsoft Azure B2C? (Answer 53).

How do I do that?

Sep 25 2024, 12:45
valerio.bozzolan added a comment to D25823: arc browse: add support to Subversion repos.

Note that I introduced the method isBranchJustFilesystemPath() that returns something different for Subversion, just to recognize Subversion,

Sep 25 2024, 09:44
valerio.bozzolan moved T15054: Add "Group by Object" in Feed search filters from 🔥 Trap to Code Sprint Candidate on the User-valerio.bozzolan board.
Sep 25 2024, 09:39 · User-valerio.bozzolan
valerio.bozzolan updated the task description for T15054: Add "Group by Object" in Feed search filters.
Sep 25 2024, 09:35 · User-valerio.bozzolan
valerio.bozzolan renamed T15054: Add "Group by Object" in Feed search filters from Improve Feed search filters to hide "minor activities" to Add "Group by Object" in Feed search filters.
Sep 25 2024, 09:34 · User-valerio.bozzolan

Sep 24 2024

valerio.bozzolan added a comment to Q35: Does Phorge provide single sign-on to Microsoft Azure B2C? (Answer 53).

(Please mark the file as visible to everybody - thanks)

Sep 24 2024, 09:53
s.quinn added a comment to Q35: Does Phorge provide single sign-on to Microsoft Azure B2C? (Answer 53).

We have a working solution. Attached. Problem is it needs the BaseURI hard coding (replaced ours with XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX in the attached code). Can anyone merge this into the baseline code?

Sep 24 2024, 09:39
s.quinn asked Q153: php-fpm memory leak.
Sep 24 2024, 07:59
Wikivu accepted rPa741f5d65c10: Change some instances of "phabricator" to "phorge".
  1. List Item
Sep 24 2024, 05:40
Wikivu awarded rP9cd62bdcc432: Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier a Orange Medal token.
Sep 24 2024, 05:35
Wikivu accepted rP9cd62bdcc432: Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier.
Sep 24 2024, 05:35

Sep 22 2024

valerio.bozzolan added a comment to D25825: Validate some user provided calendar query range dates.

Nice test plan thanks

Sep 22 2024, 18:47

Sep 20 2024

aklapper closed T15936: PHP8.1 strlen(): Passing null to parameter is deprecated, a subtask of T15064: Make Phorge compatible with PHP 8.1/8.2/8.3, as Resolved.
Sep 20 2024, 06:49 · PHP 8 support
aklapper closed T15936: PHP8.1 strlen(): Passing null to parameter is deprecated as Resolved by committing rP9cd62bdcc432: Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier.
Sep 20 2024, 06:49 · PHP 8 support
aklapper closed D25821: Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier.
Sep 20 2024, 06:49
aklapper committed rP9cd62bdcc432: Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier.
Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier
Sep 20 2024, 06:49
valerio.bozzolan accepted D25821: Fix PHP 8.1 "strlen(null)" exception in SVN commit query without identifier.

sgtm

Sep 20 2024, 02:00
valerio.bozzolan edited the content of Next Up.
Sep 20 2024, 01:59
valerio.bozzolan closed T15489: Diffusion commit feed: avoid to repeat the commit message twice as Resolved by committing rP9acdd888930e: Diffusion commit feed: fix commit title repeated twice.
Sep 20 2024, 01:58 · UX, User-valerio.bozzolan, Diffusion
valerio.bozzolan closed T15489: Diffusion commit feed: avoid to repeat the commit message twice, a subtask of T15924: Commit feed body: it should probably show (part of) commit body, as Resolved.
Sep 20 2024, 01:58 · Discussion Needed, Diffusion
valerio.bozzolan closed D25824: Diffusion commit feed: fix commit title repeated twice.
Sep 20 2024, 01:58
valerio.bozzolan committed rP9acdd888930e: Diffusion commit feed: fix commit title repeated twice.
Diffusion commit feed: fix commit title repeated twice
Sep 20 2024, 01:58

Sep 19 2024

holmboe added a watcher for arc-vscode: holmboe.
Sep 19 2024, 09:42

Sep 18 2024

sirocyl closed Q152: Tips on setting up a Phorge instance to track its own live code? as resolved.
Sep 18 2024, 19:45