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

fix: correct 1.1.2 upgrade #52

Merged
merged 1 commit into from
Nov 8, 2021
Merged

fix: correct 1.1.2 upgrade #52

merged 1 commit into from
Nov 8, 2021

Conversation

Multipage
Copy link
Contributor

{$module::MODULE_NAME} in heredoc crashes. Tested it in PHP 7.3.31 haven't tested other PHP versions but using your syntax it's crashing.
That's why i created the var $moduleName. Tested various other ways to use the $module::MODULE_NAME in the heredoc but all fail.

Preg_match and strpos was on an array $record instead of string $record->name

{$module::MODULE_NAME} in heredoc crashes. Tested it in PHP 7.3.31 haven't tested other PHP versions but using your syntax it's crashing. 
That's why i created the var $moduleName. Tested various other ways to use the $module::MODULE_NAME in the heredoc but all fail.

Preg_match and strpos was on an array $record instead of string $record->name
@EdieLemoine
Copy link
Contributor

Looks good, thank you! We'll test this, then it can be merged.

@EdieLemoine EdieLemoine changed the title Upgrade 1.1.2 fixes fix: correct 1.1.2 upgrade Nov 8, 2021
@EdieLemoine EdieLemoine merged commit 3a93fff into myparcelnl:develop Nov 8, 2021
EdieLemoine pushed a commit that referenced this pull request Nov 8, 2021
- fix syntax error
- fix carrier configuration not being migrated correctly for DPD and PostNL in MyParcel BE
RichardPerdaan pushed a commit that referenced this pull request Nov 11, 2021
- fix syntax error
- fix carrier configuration not being migrated correctly for DPD and PostNL in MyParcel BE
github-actions bot pushed a commit that referenced this pull request Nov 11, 2021
## [1.3.0](v1.2.0...v1.3.0) (2021-11-11)

### ✨ New Features

* add track trace in default prestashop field ([a274f3f](a274f3f))
* export region field ([#58](#58)) ([0950fae](0950fae))

### 🐛 Bug Fixes

* add second address line to first address line for delivery options ([#61](#61)) ([0ed533b](0ed533b))
* allow insurance options only for package type package ([#62](#62)) ([af7497c](af7497c))
* correct 1.1.2 upgrade ([#52](#52)) ([4c33988](4c33988))
* dpz weight classes automatically selected and retained ([#53](#53)) ([e398582](e398582))
* fix delivery options in frontend not loading fully sometimes ([#54](#54)) ([2ef3009](2ef3009))
* fix delivery options not being persisted in ps 1.7.8.0 ([#57](#57)) ([d625781](d625781)), closes [#49](#49)
* fix large format and return not being exported ([#64](#64)) ([17c1e4d](17c1e4d))
* fix track trace emails not being sent ([#66](#66)) ([dcebc16](dcebc16))
* fix user agent not being sent correctly ([#65](#65)) ([801f00b](801f00b))
* fix user agents not being sent ([607adc9](607adc9))
* make possible to export without insurance from order detail page ([2610e67](2610e67))
* migration will be to version 1.3.0 ([199a078](199a078))
* remove hard coded carrier from email templates and improve language ([23595c9](23595c9))
* set nextDeliveryDate when deliveryOptions are empty ([#47](#47)) ([20ab7a7](20ab7a7))
* shipment status order for digital stamp ([20dbf91](20dbf91))
* update orderlabel status during export ([03a7690](03a7690))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants