Made requested changes to category helper.

At part 3 resulting thumbnail size was not 100x100, but was 120x120.
At part 4 resulting thumbnail size was not 75x100, but was 90x120 (screenshot added for this case).

Reverted change of Enabled field name.

All works properly, but test plan may be corrected:
Part 1, 13 - may be "confirm, that in Console 3 messages about change event appeared (for 2 option checkboxes + select all itself)" - because by previous part of the plan 2 options were unchecked.

Tested with system log enabled. No deprecation warnings found. All system log messages, got when testing, displayed on screenshot -

All tests passed, excepting creating records about deprecated calls in the System Log - on tests such records never created. System Log is empty.

All right, but when upgrading from 5.1.x to 5.2.x with debugger enabled, got lot of error messages on all installation steps from 1st to 6th. Like "Table SystemLog doesn't exist"

Today made locally all requested changes, left testing by plan.

Tested with image

Tested also by D307 plan.

Test failed in the Part 4 "confirm, that image preview is displayed". Image preview not displayed. Image uploaded in the /system/images/pending/ folder, and with "/system/images/pending/{image file name}" value in DB.

Some issues on test plan:

Today tested parts 1..3 only.

Made upgrade SQL optimization.

Made all necessary fixes.

Made all requested fixes.

Filled AddedOn date in themes table.

Set AddedOn values for themes on install/upgrade.

Made all requested fixes.

Made all requested CS and logic fixes.

Made all requested changes.

today made all code changes, but not tested by plan yet

Last test in Part2 not passed. Got validation errors "bad_date_format" on CreatedOn field.

There were much of SQL errors like

Test plan must be fixed. "Enable Debug Mode in /system/debug.php file" must be added somewhere between Part 2 and Part 3. Otherwise at Part 3 Debug Mode is disabled and confirmation about appearance of notices is not possible - notices does not appear.

Tasted also on https://www.browserstack.com (IE8 part)

Tested all with IE10. Tested with IE8, as browser mode of IE10 (through IE8 mode selection in developer tools).

Today made all necessary changes locally, started writing additions to the test plan.

Today only re-applied differential patch and re-installed local version of 5.3.x In-Portal with this patch.

Patch rebuilt for 5.2.x version.

Today looks that made all necessary changes, but not full-tested yet. Left part of tests planned on next day.

Rebuilt in In-Portal 5.2.x.

Not confirmed test Part2, in Admin Console, item N4 "in heading: parent section icon and name, and on next row extra heading text". In fact, extra heading text is on top, section header - next line after extra heading text - in reverse order from defined in test plan.

Fixed CS errors.

Fixed class name in the @var type comment

There were errors when applying patch (through arc patch D253 command). Some changes not apply automatically. Applied manually before tests.

Detected bug, when one admin user opens to edit category1 with relatively big TreeLeft/TreeRight numbers. Then second admin user moves subtree, where category1 is a leaf to the zone of smaller TreeLeft/TreeTight numbers. Then, first user saves category1 and old, big TreeLeft/TreeRight numbers becomes saved to the zone of small TreeLeft/TreeRight numbers, breaking Treeleft BETWEEN A AND B logic.

Mentioned above not a bug. Clearing editor area after each step helps to avoid link merging problem.

When editing inline block in adm.console, button "Remove Link" is not available, and this makes problematic to input non-linked text after some link is added to empty editor.

Today partially tested part of part 1. Only actions for leaf operation with enabled "Recycle Bin". No bugs found.