Rebuilding patch to fix conflicts during apply.

Build #119 finished with UNSTABLE status: http://ci.in-portal.org/job/in-portal-52x/119/

Build #118 finished with UNSTABLE status: http://ci.in-portal.org/job/in-portal-52x/118/

In D315#6193, @erik wrote:

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.

Second script in the preparations part may be fixed by removing first 3 symbols "$('", that are excessive and produces js error.

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.

The deprecation notices are triggered only in Debug Mode. After talking with @erik over Skype we've found out that debug mode was disabled via debug_off cookie. Please review again.

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

After speaking with @erik it turned out, that System Log was disabled on his In-Portal install. So need to retest all over again.

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

Build #117 finished with UNSTABLE status: http://ci.in-portal.org/job/in-portal-52x/117/

In D314#6117, @erik wrote:

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"

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"

Build #50 finished with UNSTABLE status: http://ci.in-portal.org/job/in-portal-53x/50/

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

I've updated test plan, because code was fine, but test plan had some copy/paste issues.

Need to specify 'direct_links' => false was the only reason tests have failed?

Tested with image

Build #116 finished with UNSTABLE status: http://ci.in-portal.org/job/in-portal-52x/116/

Test plan updated to compensate for the found error. Turns out that include_path field option is mandatory, when kPictureFormatter is used without upload_dir option.

Tested also by D307 plan.

Please test again, because last time you've forgot to test all (skipped 1st line of test 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.

In D301#5975, @erik wrote:

Some issues on test plan:

9.5.C. - No clickable General tab on label editing screen. Button Save used for test.
11.4 - no Label field in the theme editing form (Name field is there)
20.6 - Article excerpt do not use standard inp_edit_textarea control, so it can't be tested this way without some additional changes.

Some issues on test plan:

Today tested parts 1..3 only.

Made upgrade SQL optimization.