Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev/core#3952 Add in Upgrade script to fix up print labels where " wa… #24872

Merged
merged 4 commits into from
Nov 2, 2022

Conversation

seamuslee001
Copy link
Contributor

…s not being properly escaped for json purposes on previous upgrade process

Overview

This aims to replicate the fix that was put in for new installs in #24695 but this does this for sites that had upgraded using hte previous upgrade step.

Before

Print labels in already installed sites may not be able to be json_decoded properly due to date upgrade issue

After

Should be fixed now

ping @eileenmcnaughton @totten @demeritcowboy

…s not being properly escaped for json purposes on previous upgrade process
@civibot
Copy link

civibot bot commented Nov 1, 2022

No issue was found matching the number given in the pull request title. Please check the issue number.

@civibot
Copy link

civibot bot commented Nov 1, 2022

(Standard links)

@eileenmcnaughton
Copy link
Contributor

Happily some other testing I've been doing means my db is already in the 'pre' state for this PR

image

Hmm - but I think the function version needs to change

image

@eileenmcnaughton
Copy link
Contributor

testing

  • public function upgrade_5_55_0($rev): void {
  • public function upgrade_5_55_alpha2($rev): void {
    $this->addTask(ts('Fix Event Badge Upgrade'), 'fix_event_badge_upgrade');
    $this->addTask(ts('Upgrade DB to %1: SQL', [1 => $rev]), 'runSql', $rev);

@eileenmcnaughton
Copy link
Contributor

no - it wasn't that easy

image

@eileenmcnaughton
Copy link
Contributor

Looks like it ran the update though....

image

@eileenmcnaughton
Copy link
Contributor

Same issue in the test run

Dropping SQL triggers...
Preparing upgrade...

In Form.php line 509:

[CRM_Core_Exception]
Malformed upgrade sequence. The incremental update 5.55.0 exceeds target v
ersion 5.55.beta1

Exception trace:
at /home/jenkins/bknix-dfl/build/core-24872-1wqd2/web/sites/all/modules/civicrm/CRM/Upgrade/Form.php:509
CRM_Upgrade_Form::buildQueue() at phar:///home/jenkins/bknix-dfl/bin/cv/src/Command/UpgradeDbCommand.php:117
Civi\Cv\Command\UpgradeDbCommand->execute() at phar:///home/jenkins/bknix-dfl/bin/cv/vendor/symfony/console/Command/Command.php:255

@seamuslee001 seamuslee001 merged commit a14f548 into civicrm:5.55 Nov 2, 2022
@seamuslee001 seamuslee001 deleted the dev_core_3952 branch November 2, 2022 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants