Page MenuHomePhorge
Feed Advanced Search

Oct 5 2023

valerio.bozzolan added a cover image to T15651: Tab Panel: the first Tab is not always auto-selected anymore.
Oct 5 2023, 06:38 · Dashboards
valerio.bozzolan triaged T15651: Tab Panel: the first Tab is not always auto-selected anymore as Normal priority.
Oct 5 2023, 06:38 · Dashboards
valerio.bozzolan added inline comments to rPcbc0e661544a: Change erroneous loose comparison to strict comparison in tab panels.
Oct 5 2023, 05:56

Oct 3 2023

valerio.bozzolan added a comment to Q77: Arcanist writes cache files to its source directory, and this prevents packaging.

Just for my curiosity, what lovely package are you crafting? APT? snap? Thanks :)

Oct 3 2023, 14:15 · Arcanist
valerio.bozzolan added Q77: Arcanist writes cache files to its source directory, and this prevents packaging (Answer 105).
Oct 3 2023, 14:14
valerio.bozzolan added a member for Trusted Contributors: fgaz.
Oct 3 2023, 12:56
valerio.bozzolan accepted D25445: Update path in startup error.

I accept but please wait at least one day before landing :) Thanks for your code contribution!

Oct 3 2023, 12:55
valerio.bozzolan added a comment to D25445: Update path in startup error.

I'm inclined to accept this, premising that usually we adopt PlatformSymbols::getPlatformServerPath() but in this case that will not work since this error is very low-level about the load of Arcanist itself.

Oct 3 2023, 12:54
valerio.bozzolan added a comment to T15322: Footer: specify a default Libre license for Phorge contents.

Thanks but I don't know where that typo is, so, feel free to fix

Oct 3 2023, 09:07 · Discussion Needed, Governance
valerio.bozzolan triaged T15649: Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected as Unbreak Now! priority.

Triaging to "Unbreak Now!" so to be kindly shown in the homepage :)

Oct 3 2023, 07:30 · Bug Reports
valerio.bozzolan added a comment to T15322: Footer: specify a default Libre license for Phorge contents.

I don't remember but that was an attempt to do not release Passphrase credentials as CC BY-SA 4.0 :D :D

Oct 3 2023, 07:26 · Discussion Needed, Governance
valerio.bozzolan added a comment to T15322: Footer: specify a default Libre license for Phorge contents.

Updated to reflect some tips from comments. Also added some "Pro / Cons"

Oct 3 2023, 06:10 · Discussion Needed, Governance
valerio.bozzolan updated the task description for T15322: Footer: specify a default Libre license for Phorge contents.
Oct 3 2023, 06:09 · Discussion Needed, Governance
valerio.bozzolan moved T15515: Blur profile image when account disabled from Backlog to Feature Requests on the Affects-Wikimedia board.
Oct 3 2023, 05:31 · Feature Requests, Affects-Wikimedia
valerio.bozzolan moved T15648: Transaction text when changing Diffusion URI I/O type is about Display type instead from Backlog to Bug Reports on the Affects-Wikimedia board.
Oct 3 2023, 05:31 · Affects-Wikimedia, Bug Reports
valerio.bozzolan moved T15149: Fix middle-click, CTRL+click, right-click etc. on Typehead search results from Backlog to Bug Reports on the Affects-Wikimedia board.
Oct 3 2023, 05:31 · Affects-Wikimedia, User-valerio.bozzolan, Typeahead

Oct 2 2023

valerio.bozzolan updated the test plan for D25444: Fix regression in DiffusionDiffQueryConduitAPIMethod.
Oct 2 2023, 14:52
valerio.bozzolan requested review of D25444: Fix regression in DiffusionDiffQueryConduitAPIMethod.
Oct 2 2023, 14:51
valerio.bozzolan added a revision to T15649: Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected: D25444: Fix regression in DiffusionDiffQueryConduitAPIMethod.
Oct 2 2023, 14:51 · Bug Reports
valerio.bozzolan updated the task description for T15649: Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected.
Oct 2 2023, 14:30 · Bug Reports
valerio.bozzolan raised a concern with rP69c64c1e83e2: Teach Commit View about Encoding.
Oct 2 2023, 14:30
valerio.bozzolan updated the task description for T15649: Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected.
Oct 2 2023, 14:28 · Bug Reports
valerio.bozzolan created T15649: Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected.
Oct 2 2023, 14:28 · Bug Reports
valerio.bozzolan updated the diff for D25416: Replacing the deprecated -moz-outline-style with outline-style.

git rebase master

Oct 2 2023, 11:47

Sep 29 2023

valerio.bozzolan added a comment to D25437: Fix "Undefined index" exception setting Meme text.

There is a thing, to be honest. We are trimming the $above just in the test, but then we are not using the trimmed string in the rendering. So, the $v = coalesce($v, '') thing, and always trim() later, may still be better.

Sep 29 2023, 07:58
valerio.bozzolan updated the test plan for D25437: Fix "Undefined index" exception setting Meme text.
Sep 29 2023, 07:57
valerio.bozzolan accepted D25437: Fix "Undefined index" exception setting Meme text.

Maybe other people would like to propose a coalesce() and always trim() that and just have the if(phutil_nonempty...($)), but to be honest this is just correct and somehow minimal.

Sep 29 2023, 07:54

Sep 28 2023

valerio.bozzolan added a comment to T15640: Exception: "Call to phutil_nonempty_string() expected null or a string, got: int" in PhabricatorRepository.php.

I tried browsing that page without being able to trigger the issue

Sep 28 2023, 11:31 · Affects-Wikimedia

Sep 27 2023

valerio.bozzolan awarded D25443: Correct Transaction text when changing Diffusion URI I/O type a Like token.
Sep 27 2023, 20:47
valerio.bozzolan updated the summary of D25443: Correct Transaction text when changing Diffusion URI I/O type.
Sep 27 2023, 20:46

Sep 26 2023

valerio.bozzolan added a comment to T15256: Diffusion file: add a way to copy permalink.

Yeah feel free to review D25301

Sep 26 2023, 06:55 · User-valerio.bozzolan, Diffusion

Sep 25 2023

valerio.bozzolan added a comment to D25033: Packages: Load'em from .cache.

(I have not tried this patch but I checked with ls -la the mess in my .cache to understand best practices from other software)

Sep 25 2023, 10:14 · Arcanist (archived)
valerio.bozzolan edited the content of Next Up.
Sep 25 2023, 06:23
valerio.bozzolan edited the content of Next Up.
Sep 25 2023, 06:20
valerio.bozzolan closed T15143: Allow to quick-edit a Milestone name from Workboard menu as Resolved by committing rP903d71e67d84: Workboard: Milestone Name easily editable (instead of surfing 3 pages).
Sep 25 2023, 06:18 · UX, Affects-Wikimedia, User-valerio.bozzolan, Workboard
valerio.bozzolan closed D25066: Workboard: Milestone Name easily editable (instead of surfing 3 pages).
Sep 25 2023, 06:18
valerio.bozzolan committed rP903d71e67d84: Workboard: Milestone Name easily editable (instead of surfing 3 pages).
Workboard: Milestone Name easily editable (instead of surfing 3 pages)
Sep 25 2023, 06:18
valerio.bozzolan added a comment to D25066: Workboard: Milestone Name easily editable (instead of surfing 3 pages).
  • we have an approval from a core contributor
  • we tested by at lest 2 persons in real-world
  • we tested also in my production for 7 months, using this intensively, without any nuclear implosion
  • nobody cares, really, nobody uses Milestones, nobody knows what a Milestone is, people also don't like Workboards, Workboards are evil persons for much core contributors, so we will never receive more feedback here automagically, if we have not received these in 7 months
  • since this change is online since the 11st of February (!) :D and since I waited 2 weeks to be kind after approval, ...
Sep 25 2023, 06:16
valerio.bozzolan updated the task description for T15535: Using Differential with plain Git, without requiring Arc.
Sep 25 2023, 06:01 · Feature Requests

Sep 22 2023

waldyrious awarded T15510: Make email clear about "Someone accepted revision" vs "Revision is ready to land" a Like token.
Sep 22 2023, 20:23 · UX, Differential

Sep 20 2023

valerio.bozzolan added inline comments to D25439: Fix strlen(null) errors for projects with integer fields under PHP 8.1.
Sep 20 2023, 20:28
valerio.bozzolan added a comment to D25439: Fix strlen(null) errors for projects with integer fields under PHP 8.1.

I'm generally scared by strlen() things since it means it was probably relying on implicit cast-to-string.

Sep 20 2023, 20:25
valerio.bozzolan added a comment to T15510: Make email clear about "Someone accepted revision" vs "Revision is ready to land".

I'm trying to imagine a title that also involves the UX in general about accepting revisions, since it's also about the "Accept Revision" button, but my brain is not helping me right now in this

Sep 20 2023, 15:43 · UX, Differential
valerio.bozzolan updated the task description for T15510: Make email clear about "Someone accepted revision" vs "Revision is ready to land".
Sep 20 2023, 15:42 · UX, Differential

Sep 19 2023

valerio.bozzolan added a comment to D25400: Fix a PHP 8.1 deprecated use of strlen with a NULL argument.

I was trying to test this, but I don't know how to do a Subversion commit without a committer name 🤔

Sep 19 2023, 12:34
valerio.bozzolan added inline comments to D25425: Remove mention of Phabricator in the Auth setup check.
Sep 19 2023, 12:31
valerio.bozzolan added inline comments to D25425: Remove mention of Phabricator in the Auth setup check.
Sep 19 2023, 12:29
valerio.bozzolan added inline comments to D25051: Destroy file attachments when file is deleted, or object is deleted.
Sep 19 2023, 11:03 · Files
valerio.bozzolan updated the test plan for D25066: Workboard: Milestone Name easily editable (instead of surfing 3 pages).
Sep 19 2023, 09:50

Sep 18 2023

valerio.bozzolan triaged T15639: Exception: Value passed to "phutil_string_cast()" is an array in PhabricatorApplicationSearchEngine.php as Low priority.

Decreasing just a bit the priority since we are a bit confused about how to proceed but feel free to change

Sep 18 2023, 16:18
valerio.bozzolan added a comment to T15639: Exception: Value passed to "phutil_string_cast()" is an array in PhabricatorApplicationSearchEngine.php.

It could also be some bots sending you fuzzy input. Consider to mark as invalid, until a real human being elevates this as a problem affecting some workflows.

Sep 18 2023, 09:48
valerio.bozzolan added a comment to T15639: Exception: Value passed to "phutil_string_cast()" is an array in PhabricatorApplicationSearchEngine.php.

Note that the upstream solution from Phabricator does not fix your problem, since

Sep 18 2023, 09:46
valerio.bozzolan added inline comments to D25051: Destroy file attachments when file is deleted, or object is deleted.
Sep 18 2023, 08:07 · Files
valerio.bozzolan moved T15143: Allow to quick-edit a Milestone name from Workboard menu from Backlog to Feature Requests on the Affects-Wikimedia board.
Sep 18 2023, 06:12 · UX, Affects-Wikimedia, User-valerio.bozzolan, Workboard
valerio.bozzolan added a project to T15640: Exception: "Call to phutil_nonempty_string() expected null or a string, got: int" in PhabricatorRepository.php: Affects-Wikimedia.
Sep 18 2023, 06:12 · Affects-Wikimedia
valerio.bozzolan added a comment to T15640: Exception: "Call to phutil_nonempty_string() expected null or a string, got: int" in PhabricatorRepository.php.

Are you aware of any non-git repository in Wikimedia?

Sep 18 2023, 06:11 · Affects-Wikimedia
valerio.bozzolan requested changes to D25419: Fix the whitespace issue in the message "Access Denied: Restricted File".

Kindly marking as "request changes" only to indicate "waiting for author feedback"

Sep 18 2023, 06:05 · UX

Sep 14 2023

valerio.bozzolan updated the answer details for Q75: wiki (phriction) page doesn't utilize full screen width like most other pages do (Answer 102).
Sep 14 2023, 19:52
valerio.bozzolan updated the answer details for Q75: wiki (phriction) page doesn't utilize full screen width like most other pages do (Answer 102).
Sep 14 2023, 19:52
valerio.bozzolan updated the answer details for Q75: wiki (phriction) page doesn't utilize full screen width like most other pages do (Answer 102).
Sep 14 2023, 19:51
valerio.bozzolan added Q75: wiki (phriction) page doesn't utilize full screen width like most other pages do (Answer 102).
Sep 14 2023, 19:50

Sep 13 2023

valerio.bozzolan retitled D25066: Workboard: Milestone Name easily editable (instead of surfing 3 pages) from Workboard: Milestone Name changable in 4 clicks instead of 6
Sep 13 2023, 08:40

Sep 11 2023

valerio.bozzolan added inline comments to D25435: Fix rendering of cowsay sheep.cow.
Sep 11 2023, 10:48
valerio.bozzolan added a comment to Q76: How can I display a checklist to users depending on the current status and type of the ticket? (Answer 101).

I'm talking about Herald Webhook, not Herald Rule. So you can write some web code that is visited when something happens, so you can run custom code when that happens, and you can edit your Tasks.

Sep 11 2023, 09:48
valerio.bozzolan updated the answer details for Q76: How can I display a checklist to users depending on the current status and type of the ticket? (Answer 101).
Sep 11 2023, 07:31
valerio.bozzolan added Q76: How can I display a checklist to users depending on the current status and type of the ticket? (Answer 101).
Sep 11 2023, 07:30

Sep 10 2023

valerio.bozzolan accepted D25432: Fix PHP 8.1 auth view strlen(null) error.

I tried to test this but I have not a real JIRA account and I was not able to test a non-null value. But, I inspected all the definitions of that method and it seems it only returns pht() so I would assume that it only returns strings.

Sep 10 2023, 00:27
valerio.bozzolan updated the test plan for D25432: Fix PHP 8.1 auth view strlen(null) error.
Sep 10 2023, 00:19

Sep 9 2023

valerio.bozzolan added a comment to T15632: PHP 8.1 "strlen(null)" exception rendering task with custom date field after adding project.

I think this is apparently unrelated to projects. It's just needed to edit something else, keeping the date field as empty.

Sep 9 2023, 18:33 · PHP 8 support
valerio.bozzolan retitled D25431: Fix PHP 8.1 "strlen(null)" exception rendering Task with empty custom date field from Fix PHP 8.1 "strlen(null)" exception rendering task with custom date field after adding project
Sep 9 2023, 18:32
valerio.bozzolan updated the test plan for D25431: Fix PHP 8.1 "strlen(null)" exception rendering Task with empty custom date field.
Sep 9 2023, 18:32
valerio.bozzolan accepted D25431: Fix PHP 8.1 "strlen(null)" exception rendering Task with empty custom date field.

Tested. Thanks!

Sep 9 2023, 18:31
valerio.bozzolan added a comment to D25419: Fix the whitespace issue in the message "Access Denied: Restricted File".

What do you think about the related tip? Also here I can maybe try to propose a counter-patch next week

Sep 9 2023, 14:55 · UX
valerio.bozzolan added a comment to D25390: Fix PHP 8.1 "json_decode(null)" exception editing a form when custom field of type Users exists.

What do you think about? Maybe I can try to propose a counter-patch about the related tip

Sep 9 2023, 14:54

Sep 7 2023

valerio.bozzolan accepted D25434: Add documentation for cowsay.

Yeah why not, thanks

Sep 7 2023, 08:19

Sep 6 2023

valerio.bozzolan edited the content of Planning Meetings.
Sep 6 2023, 11:33 · phorge.it install
valerio.bozzolan updated the diff for D25066: Workboard: Milestone Name easily editable (instead of surfing 3 pages).

comments more sexier for reviewers

Sep 6 2023, 11:24
valerio.bozzolan added inline comments to D25421: Audit Feed: less verbose when the author is the committer.
Sep 6 2023, 11:09
valerio.bozzolan accepted D25433: Fix Diviner strlen(null) error when clicking on a link.

The AphrontRequest::getStr() method is known to return null or a string and any other value will cause a nuclear implosion from phutil_nonempty_string() and we like that

Sep 6 2023, 10:45
valerio.bozzolan added a comment to D25434: Add documentation for cowsay.

Nice! Just few inline tips

Sep 6 2023, 08:26
valerio.bozzolan added a comment to T15401: Make the Remarkup Help Page Extensible.

While it could be nice to have this, in the meanwhile I think that having something minimal like D25434 would be nice

Sep 6 2023, 08:25 · Documentation, Remarkup
valerio.bozzolan edited the content of Next Up.
Sep 6 2023, 08:19

Sep 5 2023

valerio.bozzolan added a comment to D25434: Add documentation for cowsay.

Just a question. Why under differential and not related to Remarkup?

Sep 5 2023, 16:35
valerio.bozzolan added a comment to D25419: Fix the whitespace issue in the message "Access Denied: Restricted File".

Nice catch

Sep 5 2023, 14:52 · UX

Sep 4 2023

valerio.bozzolan closed D25413: Show a link to the Phorge homepage.
Sep 4 2023, 11:31
valerio.bozzolan committed rHP8e19e94e3b6f: Show a link to the Phorge homepage.
Show a link to the Phorge homepage
Sep 4 2023, 11:31
valerio.bozzolan added a comment to T15322: Footer: specify a default Libre license for Phorge contents.

Content is not in something, rather it's licensed under.

Sep 4 2023, 11:27 · Discussion Needed, Governance
valerio.bozzolan updated the task description for T15322: Footer: specify a default Libre license for Phorge contents.
Sep 4 2023, 11:07 · Discussion Needed, Governance
valerio.bozzolan added a comment to T15322: Footer: specify a default Libre license for Phorge contents.

Thanks @aklapper, feel free to edit the Task description according to your proposal.

Sep 4 2023, 11:04 · Discussion Needed, Governance
valerio.bozzolan accepted D25415: Make InterpreterBlockRule regex only match on valid interpreter names.

Nice catch. I agree that we should allow to have texts like asd {{{ lol }}} whey without any nuclear implosion needed by design just because we are trying to give support to cowsay or figlet.

Sep 4 2023, 10:51

Sep 1 2023

valerio.bozzolan awarded Blog Post: August 2023 Release a Love token.
Sep 1 2023, 09:16

Aug 31 2023

valerio.bozzolan updated the summary of D25413: Show a link to the Phorge homepage.
Aug 31 2023, 20:47
valerio.bozzolan awarded Q74: Minio over HTTP (without SSL) as an S3 backend a Like token.
Aug 31 2023, 11:33 · Phorge
valerio.bozzolan updated the diff for D25413: Show a link to the Phorge homepage.
  • rollback CSS change
Aug 31 2023, 08:45
valerio.bozzolan updated the summary of D25413: Show a link to the Phorge homepage.
Aug 31 2023, 08:44
valerio.bozzolan added a comment to D25413: Show a link to the Phorge homepage.

Better now?

Aug 31 2023, 08:44
valerio.bozzolan updated the diff for D25413: Show a link to the Phorge homepage.
  • integrate tips from reviewers
Aug 31 2023, 08:43
valerio.bozzolan updated the name of F344028: Phorge home proposal.png from "Screenshot_2023_08_31_104248.png" to "Phorge home proposal.png".
Aug 31 2023, 08:43
valerio.bozzolan added a comment to T15049: Segoe UI Symbol provides bad user experience.

Just to inform that maybe a sub-task bug report will be created soon. Related discussion: https://phabricator.wikimedia.org/T333885#9132176

Aug 31 2023, 08:08 · Fonts
valerio.bozzolan triaged T15559: Exception "Failed to load file data for changeset" shown for Differential revisions containing changes to images and user being logged out as Normal priority.
Aug 31 2023, 07:46 · Bug Reports, Differential