Page MenuHomeIn-Portal Phabricator

INP-1872 - Properly handle constant absence during category cache rebuild
ClosedPublic

Authored by alex on Aug 5 2024, 2:35 AM.

Details

Test Plan
  1. in IDE:
    1. open /system/config.php file
    2. set $_CONFIG['Misc']['EnableSystemLog'] to 1
    3. set $_CONFIG['Misc']['SystemLogMaxLevel'] to 5
    4. save changes
  2. login to Admin Console
  3. go to the Logs & ReportsSystem Log section
  4. remember last system log record ID
  5. go to the Website & ContentStructure & Data section
  6. click on the Rebuild Section Cache menu item under Tools toolbar button
  7. confirm, that you need to perform the rebuild
  8. go to the Logs & ReportsSystem Log section
  9. confirm, that no new system log records were added

Diff Detail

Repository
rINP In-Portal
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alex created this revision.Aug 5 2024, 2:35 AM
alex requested review of this revision.Aug 5 2024, 2:35 AM
alex added a project: Restricted Project.Aug 5 2024, 2:35 AM
erik requested changes to this revision.Aug 5 2024, 9:12 AM

System log errors exists, but, seems, not related to the current patch. Equal errors count with patch and without patch.

All errors are

This revision now requires changes to proceed.Aug 5 2024, 9:12 AM
alex added a comment.EditedOct 26 2024, 9:50 AM
In D480#9710, @erik wrote:

System log errors exists, but, seems, not related to the current patch. Equal errors count with patch and without patch.

All errors are

The issue was addressed in the D493 and D492. Please accept this revision.

alex planned changes to this revision.Oct 26 2024, 9:51 AM
alex requested review of this revision.
erik accepted this revision.Oct 28 2024, 5:20 AM

All right, when debugger is off. When debugger os on - got about 20 "Trying to access array offset on value of type bool" notices

Patch have no influence on notice count. Same notices happens and without patch. 4 (four) such notices happens when only "go to the Website & Content → Structure & Data section".

This revision is now accepted and ready to land.Oct 28 2024, 5:20 AM
This revision was automatically updated to reflect the committed changes.