Page MenuHomePhorge
Feed Advanced Search

Apr 29 2025

aklapper retitled D25981: PHPDoc: Correct syntax of variadic function params from PHPDoc: Correct syntax of variadic functions' params
Apr 29 2025, 11:29
aklapper requested review of D25981: PHPDoc: Correct syntax of variadic function params.
Apr 29 2025, 11:28
aklapper requested review of D25980: PHPDoc: Correct syntax of variadic function params.
Apr 29 2025, 11:27
aklapper requested review of D25979: Fix invalid return values for void.
Apr 29 2025, 11:21

Apr 28 2025

aklapper updated the diff for D25972: Remove non-existing min-device-pixel-ratio CSS syntax.

git rebase master

Apr 28 2025, 20:58
aklapper updated the summary of D25978: Fix wrong argument count in numerous calls' signatures.
Apr 28 2025, 20:57
aklapper closed D25973: Correct invalid min-height CSS value.
Apr 28 2025, 20:54
aklapper committed rPdc32286e4c52: Correct invalid min-height CSS value.
Apr 28 2025, 20:54
aklapper edited the content of Next Up.
Apr 28 2025, 20:54
aklapper closed D25917: Make an IE CSS hack pass W3C CSS validator.
Apr 28 2025, 20:52
aklapper committed rPbda8034982a2: Make an IE CSS hack pass W3C CSS validator.
Apr 28 2025, 20:52
aklapper updated the diff for D25917: Make an IE CSS hack pass W3C CSS validator.

git rebase master

Apr 28 2025, 20:51
aklapper closed D25977: PHPDoc: Correct some param types.
Apr 28 2025, 20:50
aklapper committed rPa52702118e59: PHPDoc: Correct some param types.
Apr 28 2025, 20:50
aklapper committed rP3cf47d2cb6b1: Remove an unneeded condition in PHUITimelineEventView.php.
Apr 28 2025, 06:31
aklapper closed D25976: Remove an unneeded condition in PHUITimelineEventView.php.
Apr 28 2025, 06:31

Apr 27 2025

aklapper updated the diff for D25976: Remove an unneeded condition in PHUITimelineEventView.php.

oh true that! sigh I should look around a bit more

Apr 27 2025, 23:33
aklapper added a comment to T16035: Update embedded copy of Font Awesome 4.7.0.

FYI I pasted the list of the 341 different FA icons used by Phorge as of 2025-04-24 (excluding the list of all icons in PHUIIconView::getIcons()) in P50

Apr 27 2025, 17:16
aklapper created P50 FontAwesome icons used on 2025-04-24, excluding the list of all items in PHUIIconView::getIcons().
Apr 27 2025, 17:15
aklapper closed T15891: Ineffective instanceof check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine() as Resolved by committing rP35e233345cd6: Fix check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().
Apr 27 2025, 17:11
aklapper closed D25975: Fix check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().
Apr 27 2025, 17:11
aklapper committed rP35e233345cd6: Fix check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().
Apr 27 2025, 17:11
aklapper requested review of D25978: Fix wrong argument count in numerous calls' signatures.
Apr 27 2025, 16:17
aklapper added a comment to D25900: Remove the onboard "mailKey" from Paste.

Argh, PEBKAC, right. Thanks!

[acko@fedora phorge (master *$|u=)]$ ./bin/storage upgrade
Target                          Error
phabricator_paste.paste.mailKey Surplus
Apr 27 2025, 15:46
aklapper added a comment to D25900: Remove the onboard "mailKey" from Paste.

@mainframe98: Does ./bin/lipsum generate pastes work as expected for you? I'm seeing errors here:

 OOPS  Generator ("Pastes") was unable to generate an object.
#1364: Field 'mailKey' doesn't have a default value
Apr 27 2025, 12:40
aklapper requested review of D25977: PHPDoc: Correct some param types.
Apr 27 2025, 11:24
aklapper requested review of D25976: Remove an unneeded condition in PHUITimelineEventView.php.
Apr 27 2025, 10:56

Apr 26 2025

aklapper added a comment to T15891: Ineffective instanceof check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().

Now that rP7aee92b5e95cfc11059e9ff8788e5e53dc88e84d is merged (plus realizing that shouldUseMySQLSearchEngine() is only called in https://we.phorge.it/source/phorge/browse/master/src/applications/config/check/PhabricatorMySQLSetupCheck.php;7aee92b5e95cfc11059e9ff8788e5e53dc88e84d$106, plus seeing rP48a34eced28d82b77eb840d05702daffb8e3ddbb), maybe PhabricatorSearchDocument::isInnoDBFulltextEngineAvailable() could be completely removed and this class could instead check the DB software version via mysqli::get_server_info and PDO::getAttribute(PDO::ATTR_SERVER_VERSION) ?
I guess that's a topic for a separate task though...?

Apr 26 2025, 20:38
aklapper requested review of D25975: Fix check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().
Apr 26 2025, 20:34
aklapper added a revision to T15891: Ineffective instanceof check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine(): D25975: Fix check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().
Apr 26 2025, 20:34
aklapper added a comment to T15891: Ineffective instanceof check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().

Quick and dirty way to test: Insert this in a random place (output listed after the // ):

$services = PhabricatorSearchService::getAllServices();
foreach ($services as $service) {
  phlog(get_class($service));                   // 'PhabricatorSearchService'
  phlog($service->getDisplayName());            // 'MySQL'
  phlog($service->getEngine());                 // Object PhabricatorFerretFulltextStorageEngine
  if ($service->getEngine() instanceof
      PhabricatorFerretFulltextStorageEngine) {
    phlog('yesh');                              // 'yesh'
  }
}
Apr 26 2025, 20:31
aklapper updated the task description for T15891: Ineffective instanceof check in PhabricatorMySQLSetupCheck:shouldUseMySQLSearchEngine().
Apr 26 2025, 20:06

Apr 25 2025

aklapper closed T15582: Allow blocking users as Wontfix.

No clarification why disabling users is not sufficient, thus declining.

Apr 25 2025, 19:41 · Clarification Needed
aklapper updated the diff for D25972: Remove non-existing min-device-pixel-ratio CSS syntax.

git rebase master

Apr 25 2025, 15:56
aklapper closed D25974: Remove IE6/7 non-standard "_zoom" CSS workaround.
Apr 25 2025, 15:52
aklapper committed rPdfe8539c6a0f: Remove IE6/7 non-standard "_zoom" CSS workaround.
Apr 25 2025, 15:52
valerio.bozzolan awarded D25974: Remove IE6/7 non-standard "_zoom" CSS workaround a Yellow Medal token.
Apr 25 2025, 14:58
aklapper requested review of D25974: Remove IE6/7 non-standard "_zoom" CSS workaround.
Apr 25 2025, 11:26
aklapper requested review of D25973: Correct invalid min-height CSS value.
Apr 25 2025, 09:59
aklapper requested review of D25972: Remove non-existing min-device-pixel-ratio CSS syntax.
Apr 25 2025, 09:54
aklapper closed D25895: Do not create search_documentfield table; update docs.
Apr 25 2025, 08:11
aklapper committed rP7aee92b5e95c: Do not create search_documentfield table; update docs.
Apr 25 2025, 08:11
aklapper added a comment to T15056: Improve Dark Mode.

If there is an issue in Phorge then please provide steps to reproduce in Phorge. Please also strip unneeded full quotes which make comments hard to read. Thanks!

Apr 25 2025, 06:20 · User-Cigaryno, User-valerio.bozzolan, Accessibility, User-Bukkit

Apr 24 2025

aklapper edited the content of Next Up.
Apr 24 2025, 22:13
aklapper closed D25971: Remove vendor-prefixed -moz-transition CSS syntax.
Apr 24 2025, 22:12
aklapper committed rP7d6c9285c357: Remove vendor-prefixed -moz-transition CSS syntax.
Apr 24 2025, 22:12
aklapper added a comment to T16035: Update embedded copy of Font Awesome 4.7.0.

but the "Regular" Icon Style is not available in the free version and is "Pro only".

I was wrong or misinterpreted. fa-regular-400 as part of the "Free" version exists at https://github.com/FortAwesome/Font-Awesome/tree/6.x/webfonts

Apr 24 2025, 19:24
aklapper requested review of D25971: Remove vendor-prefixed -moz-transition CSS syntax.
Apr 24 2025, 19:09
aklapper edited the content of Next Up.
Apr 24 2025, 18:11
aklapper closed D25970: Remove vendor-prefixed background-clip CSS syntax.
Apr 24 2025, 18:11
aklapper committed rP3fd24004d241: Remove vendor-prefixed background-clip CSS syntax.
Apr 24 2025, 18:11
aklapper updated the diff for D25970: Remove vendor-prefixed background-clip CSS syntax.

git rebase master

Apr 24 2025, 18:10
aklapper edited the content of Next Up.
Apr 24 2025, 17:01
aklapper committed rP367de18e7902: Remove orphan CSS rules from phui-timeline-view.css.
Apr 24 2025, 17:01
aklapper closed D25918: Remove orphan CSS rules from phui-timeline-view.css.
Apr 24 2025, 17:01
aklapper requested review of D25970: Remove vendor-prefixed background-clip CSS syntax.
Apr 24 2025, 16:57
aklapper edited the content of Next Up.
Apr 24 2025, 15:46
aklapper closed D25950: Remove PHP APC support and APC references.
Apr 24 2025, 15:46
aklapper committed rP3f5f18a8d6ee: Remove PHP APC support and APC references.
Apr 24 2025, 15:46
aklapper retitled D25918: Remove orphan CSS rules from phui-timeline-view.css from Fix invalid CSS border value for .phui-timeline-preview-header
Apr 24 2025, 15:26
aklapper updated the diff for D25918: Remove orphan CSS rules from phui-timeline-view.css.

Remove two orphan CSS rules from phui-timeline-view.css

Apr 24 2025, 15:25
aklapper updated the diff for D25918: Remove orphan CSS rules from phui-timeline-view.css.

git rebase master only; still Needs Revision

Apr 24 2025, 15:09
aklapper closed T16002: Hide "Audits" section in Email Preferences when Audit is uninstalled as Resolved by committing rPe45f29d965ad: Hide "Audits" section in Email Preferences when uninstalled.
Apr 24 2025, 11:10 · Bug Reports
aklapper closed D25894: Hide "Audits" section in Email Preferences when uninstalled.
Apr 24 2025, 11:10
aklapper committed rPe45f29d965ad: Hide "Audits" section in Email Preferences when uninstalled.
Apr 24 2025, 11:10
aklapper closed D25911: Diviner: Configuring Outbound Mail: Remove incorrect Gmail references.
Apr 24 2025, 11:10
aklapper committed rP7ff426a2bdfc: Diviner: Configuring Outbound Mail: Remove incorrect Gmail references.
Apr 24 2025, 11:10
aklapper added a comment to D25911: Diviner: Configuring Outbound Mail: Remove incorrect Gmail references.

Small question: if an user migrates from gmail.com "less secure apps" to gmail.com "App Passwords", isn't that the very same thing?

Apr 24 2025, 11:09
aklapper closed T15829: Deprecation notice @ PhabricatorStandardPageView.php:630 (missing User-Agent HTTP Header), a subtask of T15064: Make Phorge compatible with PHP 8.1/8.2/8.3/8.4, as Resolved.
Apr 24 2025, 06:08 · PHP 8 support
aklapper closed T15829: Deprecation notice @ PhabricatorStandardPageView.php:630 (missing User-Agent HTTP Header) as Resolved by committing rP372316c99836: Fix PHP 8.1 "preg_match(null)" exception for missing User-Agent HTTP Header.
Apr 24 2025, 06:08 · PHP 8 support, Deprecation Notice
aklapper closed D25868: Fix PHP 8.1 "preg_match(null)" exception for missing User-Agent HTTP Header.
Apr 24 2025, 06:08
aklapper committed rP372316c99836: Fix PHP 8.1 "preg_match(null)" exception for missing User-Agent HTTP Header.
Apr 24 2025, 06:08
aklapper edited the content of Next Up.
Apr 24 2025, 06:05
aklapper closed D25919: Fix invalid CSS cursor value for .phui-oi-grippable.
Apr 24 2025, 06:05
aklapper committed rP3acb080f21a2: Fix invalid CSS cursor value for .phui-oi-grippable.
Apr 24 2025, 06:05
aklapper updated the diff for D25919: Fix invalid CSS cursor value for .phui-oi-grippable.

git rebase master

Apr 24 2025, 06:04
aklapper closed D25958: PHPDoc: Fix @return string for methods which can also return null.
Apr 24 2025, 05:59
aklapper committed rP403fb8a85214: PHPDoc: Fix @return string for methods which can also return null.
Apr 24 2025, 05:59

Apr 23 2025

aklapper closed D25953: PHPDoc: Replace invalid "key" and "value" types.
Apr 23 2025, 14:38
aklapper committed rARC9454fa21da41: PHPDoc: Replace invalid "key" and "value" types.
Apr 23 2025, 14:38
aklapper closed D25954: PHPDoc: Correct marker in one class header.
Apr 23 2025, 14:37
aklapper committed rP6a0e70d7163c: PHPDoc: Correct marker in one class header.
Apr 23 2025, 14:37
aklapper closed D25952: PHPDoc: Replace invalid "obj" with "object".
Apr 23 2025, 14:37
aklapper committed rPe31c3041faec: PHPDoc: Replace invalid "obj" with "object".
Apr 23 2025, 14:37
20after4 awarded rARC4db5dc25e4e8: PHPDoc: Replace invalid "obj" with "object" a Mountain of Wealth token.
Apr 23 2025, 14:37
aklapper closed D25951: PHPDoc: Replace invalid "obj" with "object".
Apr 23 2025, 14:36
aklapper committed rARC4db5dc25e4e8: PHPDoc: Replace invalid "obj" with "object".
Apr 23 2025, 14:36

Apr 22 2025

aklapper placed T15513: Cannot edit Conpherence rooms on mobile up for grabs.

Fix was reverted for now as it had bad side effects.

Apr 22 2025, 13:17 · User-Cigaryno, Mobile, Conpherence, Bug Reports
aklapper reopened T15513: Cannot edit Conpherence rooms on mobile as Open by committing rPcadc00d744eb: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 13:16 · User-Cigaryno, Mobile, Conpherence, Bug Reports
aklapper closed T16040: Regression: Sending a message in Conpherence rooms opens dialog to edit room as Resolved by committing rPcadc00d744eb: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 13:16 · UX, Bug Reports, Aphlict, Conpherence
aklapper closed D25969: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 13:16
aklapper added a reverting change for D25879: Fix editing Conpherence rooms on mobile: rPcadc00d744eb: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 13:16
aklapper committed rPcadc00d744eb: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 13:16
aklapper added a reverting change for rPfd6118bfa601: Fix editing Conpherence rooms on mobile: rPcadc00d744eb: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 13:16
aklapper requested review of D25969: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 12:55
aklapper added a reverting change for D25879: Fix editing Conpherence rooms on mobile: D25969: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 12:55
aklapper added a revision to T15513: Cannot edit Conpherence rooms on mobile: D25969: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 12:55 · User-Cigaryno, Mobile, Conpherence, Bug Reports
aklapper added a revision to T16040: Regression: Sending a message in Conpherence rooms opens dialog to edit room: D25969: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 12:55 · UX, Bug Reports, Aphlict, Conpherence
aklapper added a reverting change for rPfd6118bfa601: Fix editing Conpherence rooms on mobile: D25969: Revert "Fix editing Conpherence rooms on mobile".
Apr 22 2025, 12:55
aklapper renamed T16040: Regression: Sending a message in Conpherence rooms opens dialog to edit room from Fix bug with inability to take actions in Conpherence rooms as dialog to edit room appears instead to Regression: Sending a message in Conpherence rooms opens dialog to edit room.
Apr 22 2025, 12:55 · UX, Bug Reports, Aphlict, Conpherence