Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#43925 closed defect (bug) (fixed)

Inconsistencies in GDPR strings

Reported by: audrasjb's profile audrasjb Owned by: azaozz's profile azaozz
Milestone: 4.9.6 Priority: normal
Severity: normal Version: 5.1
Component: Privacy Keywords: needs-testing gdpr
Focuses: Cc:

Description

There is a bunch of small inconsistencies in the new GDPR strings.
Especially in feedback messages during data exports, sometimes we have a trailing dot, sometimes not.

This ticket can be used to track i18n inconsistencies and fixes before 4.9.6 landing.

Attachments (4)

43925.1.diff (2.1 KB) - added by audrasjb 7 years ago.
Fixes small i18n inconsistencies in wp-admin/includes/file.php
43925.2.diff (3.2 KB) - added by audrasjb 7 years ago.
Updated i18n inconsistencies fixes
43925.3.diff (7.1 KB) - added by audrasjb 7 years ago.
@presskop enhancements + some "XXX failed!" fixes
43925.4.diff (7.8 KB) - added by audrasjb 7 years ago.
adds some context to file.php strings - thanks @presskop

Download all attachments as: .zip

Change History (20)

This ticket was mentioned in Slack in #gdpr-compliance by audrasjb. View the logs.


7 years ago

#2 @audrasjb
7 years ago

Sorry, my last patch is completely wrong, I get a GIT issue. Will upload it again in few minutes.

Version 0, edited 7 years ago by audrasjb (next)

@audrasjb
7 years ago

Fixes small i18n inconsistencies in wp-admin/includes/file.php

@audrasjb
7 years ago

Updated i18n inconsistencies fixes

This ticket was mentioned in Slack in #polyglots by audrasjb. View the logs.


7 years ago

#4 @Presskopp
7 years ago

I'd prefer st. like "Email could not be send" instead of "Email Failed!"

https://core.trac.wordpress.org/browser/branches/4.9/src/wp-admin/includes/user.php?marks=1356#L1356

@audrasjb
7 years ago

@presskop enhancements + some "XXX failed!" fixes

#6 @audrasjb
7 years ago

  • Keywords needs-patch removed

Hi @Presskopp
Thank you!

In 43925.3.diff

  • @Presskopp fixes
  • replace every XXX failed! strings with more accurate sentences.

Feel free to add any other i18n issues!

Cheers,
Jb

#7 @subrataemfluence
7 years ago

  • Keywords has-patch added

#8 @audrasjb
7 years ago

  • Keywords has-patch removed

Removing has-patch keyword since we are waiting for polyglot teams feedbacks.

This ticket was mentioned in Slack in #polyglots by audrasjb. View the logs.


7 years ago

@audrasjb
7 years ago

adds some context to file.php strings - thanks @presskop

This ticket was mentioned in Slack in #gdpr-compliance by audrasjb. View the logs.


7 years ago

#12 @birgire
7 years ago

Here are some notes

https://core.trac.wordpress.org/ticket/43438#comment:43

about string consistency in wp_ajax_wp_privacy_export_personal_data() and wp_ajax_wp_privacy_erase_personal_data() that might belong here?

Last edited 7 years ago by birgire (previous) (diff)

#13 @azaozz
7 years ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In 43118:

Privacy: fix inconsistencies in new strings.

Props audrasjb.
Fixes #43925.

#14 @azaozz
7 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopen for 4.9.6.

#15 @azaozz
7 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 43119:

Privacy: fix inconsistencies in new strings.

Props audrasjb.
Merges [43118] to the 4.9 branch.
Fixes #43925.

#16 @desrosj
7 years ago

  • Component changed from General to Privacy

Moving to the new Privacy component.

Note: See TracTickets for help on using tickets.