Page MenuHomePhorge

valerio.bozzolan (Valerio Bozz.)
ʕ•ᴥ•ʔ

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Jul 6 2021, 12:36 (117 w, 1 d)
Availability
Available
Timezone
UTC+2

I'm a pasta pizza volunteer Phabrilover. I would like to do it all day long.

I use Phorge in:

  • my personal installation to handle my stuff (about 80 users)
  • in the company I work for (about 30 employees)
  • in the Wikimedia Phabricator (as simple user)

I'm not an official maintainer. I'm just an active contributor.

Please don't contact me in private. Use the chat or ping me from Ponder.

Other links:

Copyright

Since we have not a default Free license in the website (T15322), assume this Free license on my contributions:

https://creativecommons.org/licenses/by-sa/4.0/

Or, at your opinion, Apache 2.0 (same license of Phorge).

Recent Activity

Yesterday

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 :)

Tue, Oct 3, 14:15 · Arcanist
valerio.bozzolan added Q77: Arcanist writes cache files to its source directory, and this prevents packaging (Answer 105).
Tue, Oct 3, 14:14
valerio.bozzolan added a member for Trusted Contributors: fgaz.
Tue, Oct 3, 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!

Tue, Oct 3, 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.

Tue, Oct 3, 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

Tue, Oct 3, 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 :)

Tue, Oct 3, 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

Tue, Oct 3, 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"

Tue, Oct 3, 06:10 · Discussion Needed, Governance
valerio.bozzolan updated the task description for T15322: Footer: specify a default Libre license for Phorge contents.
Tue, Oct 3, 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.
Tue, Oct 3, 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.
Tue, Oct 3, 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.
Tue, Oct 3, 05:31 · Affects-Wikimedia, User-valerio.bozzolan, Typeahead

Mon, Oct 2

valerio.bozzolan updated the test plan for D25444: Fix regression in DiffusionDiffQueryConduitAPIMethod.
Mon, Oct 2, 14:52
valerio.bozzolan requested review of D25444: Fix regression in DiffusionDiffQueryConduitAPIMethod.
Mon, Oct 2, 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.
Mon, Oct 2, 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.
Mon, Oct 2, 14:30 · Bug Reports
valerio.bozzolan raised a concern with rP69c64c1e83e2: Teach Commit View about Encoding.
Mon, Oct 2, 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.
Mon, Oct 2, 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.
Mon, Oct 2, 14:28 · Bug Reports
valerio.bozzolan updated the diff for D25416: Replacing the deprecated -moz-outline-style with outline-style.

git rebase master

Mon, Oct 2, 11:47

Fri, Sep 29

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.

Fri, Sep 29, 07:58
valerio.bozzolan updated the test plan for D25437: Fix "Undefined index" exception setting Meme text.
Fri, Sep 29, 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.

Fri, Sep 29, 07:54

Thu, Sep 28

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

Thu, Sep 28, 11:31 · Affects-Wikimedia

Wed, Sep 27

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

Tue, Sep 26

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

Yeah feel free to review D2530

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

Mon, Sep 25

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)

Mon, Sep 25, 10:14 · Arcanist (archived)
valerio.bozzolan edited the content of Next Up.
Mon, Sep 25, 06:23
valerio.bozzolan edited the content of Next Up.
Mon, Sep 25, 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).
Mon, Sep 25, 06:18 · UX, Affects-Wikimedia, User-valerio.bozzolan, Workboard
valerio.bozzolan closed D25066: Workboard: Milestone Name easily editable (instead of surfing 3 pages).
Mon, Sep 25, 06:18
valerio.bozzolan committed rP903d71e67d84: Workboard: Milestone Name easily editable (instead of surfing 3 pages) (authored by valerio.bozzolan).
Workboard: Milestone Name easily editable (instead of surfing 3 pages)
Mon, Sep 25, 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, ...
Mon, Sep 25, 06:16
valerio.bozzolan updated the task description for T15535: Using Differential with plain Git, without requiring Arc.
Mon, Sep 25, 06:01 · Feature Requests

Fri, Sep 22

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

Wed, Sep 20

valerio.bozzolan added inline comments to D25439: Fix strlen(null) errors for projects with integer fields under PHP 8.1.
Wed, Sep 20, 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.

Wed, Sep 20, 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

Wed, Sep 20, 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".
Wed, Sep 20, 15:42 · UX, Differential

Tue, Sep 19

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 🤔

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

Mon, Sep 18

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

Mon, Sep 18, 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.

Mon, Sep 18, 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

Mon, Sep 18, 09:46
valerio.bozzolan added inline comments to D25051: Destroy file attachments when file is deleted.
Mon, Sep 18, 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.
Mon, Sep 18, 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.
Mon, Sep 18, 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?

Mon, Sep 18, 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"

Mon, Sep 18, 06:05

Thu, Sep 14

valerio.bozzolan updated the answer details for Q75: wiki (phriction) page doesn't utilize full screen width like most other pages do (Answer 102).
Thu, Sep 14, 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).
Thu, Sep 14, 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).
Thu, Sep 14, 19:51
valerio.bozzolan added Q75: wiki (phriction) page doesn't utilize full screen width like most other pages do (Answer 102).
Thu, Sep 14, 19:50

Wed, Sep 13

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
Wed, Sep 13, 08:40

Mon, Sep 11

valerio.bozzolan added inline comments to D25435: Fix rendering of cowsay sheep.cow.
Mon, Sep 11, 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.

Mon, Sep 11, 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).
Mon, Sep 11, 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).
Mon, Sep 11, 07:30

Sun, Sep 10

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.

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

Sat, Sep 9

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.

Sat, Sep 9, 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
Sat, Sep 9, 18:32
valerio.bozzolan updated the test plan for D25431: Fix PHP 8.1 "strlen(null)" exception rendering Task with empty custom date field.
Sat, Sep 9, 18:32
valerio.bozzolan accepted D25431: Fix PHP 8.1 "strlen(null)" exception rendering Task with empty custom date field.

Tested. Thanks!

Sat, Sep 9, 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

Sat, Sep 9, 14:55
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 ti

Sat, Sep 9, 14:54

Thu, Sep 7

valerio.bozzolan accepted D25434: Add documentation for cowsay.

Yeah why not, thanks

Thu, Sep 7, 08:19

Wed, Sep 6

valerio.bozzolan edited the content of Planning Meetings.
Wed, Sep 6, 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

Wed, Sep 6, 11:24
valerio.bozzolan added inline comments to D25421: Audit Feed: less verbose when the author is the committer.
Wed, Sep 6, 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

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

Nice! Just few inline tips

Wed, Sep 6, 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

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

Tue, Sep 5

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

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

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

Nice catch

Tue, Sep 5, 14:52

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 (authored by valerio.bozzolan).
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
valerio.bozzolan added a comment to D25420: Improving UX for ignoring timezone conflicts.

(I accepted but before landing please first review all not-done inline comments: https://we.phorge.it/differential/revision/inlines/25420/ and this comment: https://we.phorge.it/D25420#12264 )

Aug 31 2023, 05:13
valerio.bozzolan accepted D25420: Improving UX for ignoring timezone conflicts.

I like this approach, and prefer it to my simpler attempt from D25426; but have some minor suggestions:

  • Instead of "You can ignore this conflict or adjust your profile setting to match your client", we should use the same order in the sentence as the options are shown to the user: "You can adjust your profile setting to match your client, or ignore this conflict."
  • The label for the checkbox should be just "Ignore Conflict", without the "Or" at the start.
  • When the "Ignore Conflict" checkbox is selected, the timezone dropdown should be disabled (and re-enabled if the checkbox is unmarked)
Aug 31 2023, 05:09

Aug 30 2023

valerio.bozzolan marked Q72: Add support for Cloudflare Turnstile (A Captcha alternative)? (Answer 97) as hidden.
Aug 30 2023, 16:25