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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix HIX value difference between CMS and Textlab Online service #2668

Merged
merged 1 commit into from Mar 19, 2024

Conversation

seluianova
Copy link
Contributor

@seluianova seluianova commented Feb 21, 2024

Short description

It was discovered that the HIX value in https://textlab.online and in CMS is not identical:
image
image

Proposed changes

As I found, TextLab Online uses <br> for line breaks, while we used \r\n.
And that seems to be the root cause of the observed difference.

Then when I changed the line breaks in our text normalization function, I found that this actually makes the entire function unnecessary. Because it fixed all of our previous problems with the HIX value difference that I was aware of 馃檲
Including:

  • HIX value diff when adding empty lines
  • HIX value diff between pre save signal and tinymce
    etc.

Side effects

I tested everything I could think of, but I may have missed something.
So additional testing is certainly welcomed.

Resolved issues

Fixes: #


Pull Request Review Guidelines

@seluianova seluianova requested a review from a team as a code owner February 21, 2024 19:54
Copy link

codeclimate bot commented Feb 21, 2024

Code Climate has analyzed commit 03324b6 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 0.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 82.1% (0.0% change).

View more on Code Climate.

Copy link
Member

@MizukiTemma MizukiTemma left a comment

Choose a reason for hiding this comment

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

Because it fixed all of our previous problems with the HIX value difference that I was aware of 馃檲

馃檲

Thank you so much for all the analysis and work 馃檹 Looks good!

@seluianova seluianova merged commit ae42393 into develop Mar 19, 2024
5 checks passed
@seluianova seluianova deleted the bugfix/hix-text-normalization branch March 19, 2024 12:33
@timobrembeck timobrembeck mentioned this pull request Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants