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

54399 Correct handling of Umlauts and Special Characters during language switch #957

Conversation

dai-eastgate
Copy link
Contributor

@dai-eastgate dai-eastgate commented Nov 7, 2024

related to #953
changed log:

@fredericalpers fredericalpers linked an issue Nov 8, 2024 that may be closed by this pull request
5 tasks
@fredericalpers fredericalpers added bug Indicates an unexpected problem or unintended behaviour component: property detail page Issue, Pull Request or Discussion related to property detail page QA Issue or Pull request that is in review labels Nov 8, 2024
@fredericalpers fredericalpers added this to the v5.9 milestone Nov 8, 2024
@fredericalpers fredericalpers added the deploy test Triggers event to create test version label Nov 8, 2024
Copy link

github-actions bot commented Nov 8, 2024

Steps to install the approved version:

  1. Download onoffice-5.2-4-gea3ecd5f-please-unpack.zip from https://github.com/onOffice-Web-Org/oo-wp-plugin/actions/runs/11740674901.
  2. Unpack the downloaded file to get another .zip file.
  3. Upload that inner .zip file to WordPress.

@fredericalpers fredericalpers modified the milestones: v5.9, v5.3 Nov 8, 2024
@andernath andernath added deploy test Triggers event to create test version and removed deploy test Triggers event to create test version labels Nov 15, 2024
Copy link

Steps to install the approved version:

  1. Download onoffice-5.2-8-g10d06c13-please-unpack.zip from https://github.com/onOffice-Web-Org/oo-wp-plugin/actions/runs/11854003833.
  2. Unpack the downloaded file to get another .zip file.
  3. Upload that inner .zip file to WordPress.

Copy link
Contributor

@andernath andernath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dai-eastgate
Everything looks fine :), just one question. Could you please take a look?

@@ -35,6 +35,7 @@ class EstateDetailUrl
* @param string|null $title
* @param string|null $oldUrl
* @param bool $flag
* @param string|null $switchLocale
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is type null needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andernath It really isn't necessary. I'll fix it.

@andernath andernath merged commit 3590d90 into master Nov 18, 2024
2 checks passed
@andernath andernath deleted the 54399-correct-handling-of-umlauts-and-special-characters-during-language-switch branch November 18, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behaviour component: property detail page Issue, Pull Request or Discussion related to property detail page deploy test Triggers event to create test version QA Issue or Pull request that is in review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct handling of Umlauts and Special Characters during language switch
3 participants