#55758 closed enhancement (fixed)
Remove self-references ("we") in text in core, take 2
Reported by: | kebbet | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Text Changes | Keywords: | has-patch has-copy-review |
Focuses: | ui-copy | Cc: |
Description
This is a follow up ticket to #46057 which is closed and marked as fixed.
Some more instances was found. There are more of them in privacy-related strings, but I left them out from this list.
If you reach out to us, we'll also ensure you'll never have this problem again. wp-admin/includes/class-wp-automatic-updater.php:832 When you've been able to update using the "Update now" button on Dashboard > Updates, we'll clear this error for future update attempts. wp-admin/includes/class-wp-site-health-auto-updates.php:169 We'll try again with the next release. wp-admin/includes/class-wp-site-health-auto-updates.php:186 The update cannot be installed because we will be unable to copy some files. This is usually due to inconsistent file permissions. wp-admin/includes/update-core.php:1169 wp-admin/includes/update-core.php:1205 We couldn’t locate %s. Please try another nearby city. For example: Kansas City; Springfield; Portland. wp-admin/js/dashboard.js:570 There doesn't seem to be a %s file. I need this before we can get started. wp-load.php:90
Attachments (2)
Change History (18)
#1
@
2 years ago
- Keywords needs-testing needs-copy-review has-patch added; needs-patch removed
- Severity changed from minor to normal
#2
follow-up:
↓ 3
@
2 years ago
Thanks for reaching out for feedback @kebbet!
Here's some initial thoughts:
If you reach out to us, we'll also ensure you'll never have this problem again.
Reach out to WordPress Core developers to ensure you'll never have this problem again.
- While this ticket is about removing "we", I think "us" is a similar issue.
- By specifying "WordPress Core developers", this makes it more likely that users come to the right place, and don't just find "WordPress developers".
We'll try again with the next release.
Another attempt will be made with the next release.
- This is just because it's closer to the original text.
The update cannot be installed because we will be unable to copy some files. This is usually due to inconsistent file permissions.
The update cannot be installed because some files cannot be copied. This is usually due to inconsistent file permissions.
OR
The update was not installed because some files could not be copied. This is usually due to inconsistent file permissions.
- "are not possible to copy" reads a little strange to me.
- The Alternative option may be preferred to avoid using "cannot" twice.
Also pinging @marybaum for copy review. Tag, you're it!
#3
in reply to:
↑ 2
@
2 years ago
Thank you for your feedback @costdev!
Replying to costdev:
Thanks for reaching out for feedback .kebbet!
Here's some initial thoughts:
If you reach out to us, we'll also ensure you'll never have this problem again.
Reach out to WordPress Core developers to ensure you'll never have this problem again.
- While this ticket is about removing "we", I think "us" is a similar issue.
- By specifying "WordPress Core developers", this makes it more likely that users come to the right place, and don't just find "WordPress developers".
Nice suggestion, sounds better to me!
We'll try again with the next release.
Another attempt will be made with the next release.
- This is just because it's closer to the original text.
Nice suggestion too, sounds better to me!
The update cannot be installed because we will be unable to copy some files. This is usually due to inconsistent file permissions.
The update cannot be installed because some files cannot be copied. This is usually due to inconsistent file permissions.
OR
The update was not installed because some files could not be copied. This is usually due to inconsistent file permissions.
- "are not possible to copy" reads a little strange to me.
- The Alternative option may be preferred to avoid using "cannot" twice.
Not so sure about changeing from cannot (future) to was not (past), but agree double up of cannot is not preffered.
Maybe: The update cannot be installed because some files could not be copied. This is usually due to inconsistent file permissions.
?
#4
@
2 years ago
Not so sure about changeing from cannot (future) to was not (past), but agree double up of cannot is not preffered.
Excellent point. These appear to be pre-update checks that are failing.
Maybe:
The update cannot be installed because some files could not be copied. This is usually due to inconsistent file permissions.
?
These appear to be pre-update checks, so the attempt to copy the files has not yet taken place, it's just checked to see if they can be copied. therefore "could not" and my previous suggestion of "was not" don't seem appropriate here.
The update cannot be installed because we will be unable to copy some files. This is usually due to inconsistent file permissions.
What about replacing "we" with "WordPress"?
The update cannot be installed because WordPress will be unable to copy some files. This is usually due to inconsistent file permissions.
Also, I wonder, should it be "is unable" rather than "will be unable"?
i.e. "The future cannot happen because of the present situation" instead of "The future cannot happen because the future cannot happen".
If so, maybe this is more accurate:
The update cannot be installed because WordPress is unable to copy some files. This is usually due to inconsistent file permissions.
#5
follow-up:
↓ 6
@
2 years ago
Great improvement, but what about being a bit more precise:
The update cannot be installed because your WordPress site is unable to copy some files. This is usually due to inconsistent file permissions.
#6
in reply to:
↑ 5
@
2 years ago
Replying to kebbet:
Great improvement, but what about being a bit more precise:
The update cannot be installed because your WordPress site is unable to copy some files. This is usually due to inconsistent file permissions.
Maybe just "your site" instead of "your WordPress site"?
The update cannot be installed because your site is unable to copy some files. This is usually due to inconsistent file permissions.
#8
@
2 years ago
Nice improvements in accuracy and brevity here. Our latest revision in Sergey's comment is good to go IMO.
#9
@
2 years ago
- Component changed from Administration to Text Changes
- Keywords needs-copy-review removed
55758-php-2.diff
implements the reviewed strings. Thank you @costdev & @SergeyBiryukov!
The patch now includes these changes:
There doesn't seem to be a %s file. I need this before we can get started.
There doesn't seem to be a %s file. It is needed before the installation can continue.
If you reach out to us, we'll also ensure you'll never have this problem again.
Reach out to WordPress Core developers to ensure you'll never have this problem again.
When you've been able to update using the \"Update now\" button on Dashboard > Updates, we'll clear this error for future update attempts.
When you've been able to update using the \"Update now\" button on Dashboard > Updates, this error will be cleared for future update attempts.
We'll try again with the next release.
Another attempt will be made with the next release.
The update cannot be installed because we will be unable to copy some files. This is usually due to inconsistent file permissions.
The update cannot be installed because your site is unable to copy some files. This is usually due to inconsistent file permissions.
#10
@
2 years ago
55758-php-2.diff LGTM 👍
As this ticket makes string changes only, is the needs-testing
keyword accurate? Genuine question - I'm not sure what the answer is.
#11
@
2 years ago
- Keywords has-copy-review added; needs-testing removed
Good question! Handbook here: https://make.wordpress.org/core/handbook/contribute/trac/keywords/
Updates keywords in accordance to how I interpret the situation.
#13
@
2 years ago
As per above discussions and suggestions, 55758-php-2.diff looks good to me.
Thanks everyone involved.
It would be great if we can ship this one in 6.1
All suggestions for PHP-files in a combined patch.
If you reach out to us, we'll also ensure you'll never have this problem again.
If you reach out to us, the WordPress developers will also ensure you'll never have this problem again.
When you've been able to update using the "Update now" button on Dashboard > Updates, we'll clear this error for future update attempts.
When you've been able to update using the \"Update now\" button on Dashboard > Updates, this error will be cleared for future update attempts.
We'll try again with the next release.
At the next release, a new attempt will be made.
The update cannot be installed because we will be unable to copy some files. This is usually due to inconsistent file permissions.
The update cannot be installed because some files are not possible to copy. This is usually due to inconsistent file permissions.
There doesn't seem to be a %s file. I need this before we can get started.
There doesn't seem to be a %s file. It is needed before the installation can continue.