Posted by hass on May 23, 2009 at 10:38am
5 followers
| Project: | XML sitemap |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | hass |
| Status: | closed (fixed) |
Issue Summary
Patch attached with a few corrections.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| xmlsitemap_string_review-D61.patch | 6.46 KB | Ignored: Check issue status. | None | None |
Comments
#1
I will ask to my English counselor about the different English sentences proposed, which seems to have a slightly different meaning.
Only a note about invalid: there are no differences between not valid, and invalid, except that invalid also means a person made weak or disabled by illness or injury. This is personal preference, and I prefer to use not valid.
#2
Invalid for most English speakers means not valid. We very rarely use or hear the connotation for a weak person.
#3
Thanks for your contribution with this.
As you speak English as first language, do you see any differences (even if nuances) between there are no settings currently defined, and there are currently no settings defined? Or is it one grammatically correct, and one is not correct?
The definition I reported for invalid is the definition given in the New Oxford American Dictionary. If even the word would be interpreted as only meaning not valid, the path is not valid is still understood, it's grammatically correct, it's not jargon, nor it's slang. Without a valid reason to change it, I don't see why it should be changed.
#4
In Drupal core, http://api.drupal.org/api/function/path_admin_form_validate/6 uses
t("The path '@link_path' is either invalid or you do not have access to it.");so I'd say it's ok to use. I'll look over the rest of the changes.#5
The task has been partially completed.
There is another string I am not going to change. Apply cannot be changed to Save because it is used on a form where there is nothing to be save, but an operation is being applied.
So far, there is just a string that could need to be changed.
#6
BTW, there is an error with an extra single quote in the change you just made: http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/xmlsitemap/....
#7
In that case, I should adopt the same string, but I am not going to use The path is invalid..
#8
Also, in http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/xmlsitemap/..., you should not use 'The priority of a link determinates its position in the sitemap; the links with a higher priority will appear first in the sitemap.' as determinates is not an english word. Please use determines.
#9
I always confuse the adjective determinate with the verb to determine. It must be because in my language the adjective determinate doesn't exist; the word we use is the past particle of to determine (determinate comes from the Latin word that is the past participle of the Latin equivalent for to determine).
Anyway, thanks for reporting what was wrong in the commits I made; your help is well appreciated.
#10
Kiam, no problem. Just so you know, you just changed 'determinates' to 'determinetes' which is still wrong. It should be 'determines'.
http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/xmlsitemap/...
#11
I noticed there is something wrong in the files I locally have; I have fixed a problem with extra apices, but I still see a syntax error about that, which I fixed before.
#12
The inflections of the voice are different for the two. I'm not sure how to describe it. The a is softer for the made weak definition and the lid is stronger with the l belonging more to lid. The not valid definition the a is stronger and the l leans toward val. The syllables are more in-val-id for the not valid definition and in-va-lid for the made weak definition.
HTH
#13
Thanks for the clarification, earnie.
I changed that string, and used the equivalent of the string used by Drupal.
Now, the only string left is there are no settings currently defined that is equivalent to there are currently no settings defined. I asked to my English advisor, and she told me that. Being this the case, I am not going to change that string.
The task should be considerate concluded, but I will leave it open, in the case I missed something.
#14
Doesn't
Applysave a configuration setting in the database? Sorry I haven't taken a look to the UI about this. If "apply" save settings with ajax without reloading the page it's ok, but it would be the first Drupal page we have used "apply". Their is no "Apply" string in core or any other module I have installed except views... "Save" is always used.#15
I find hard to propose a different string used in a form button without to even see the relative form, and understand the context.
That form simply applies an operation that is selected by the user; it doesn't save anything. I find that save referred to an operation like submit the sitemap to the search engines sounds quite a little incorrect.
#16
I am going to change the status of the report. If there is something else to change, feel free to reopen it.
Please, reopen it for really necessary changes that are motivated; don't open it to report a change from you and me to me and you, in example.
#17
Automatically closed -- issue fixed for 2 weeks with no activity.
#18
May needs a re-role/re-apply for beta0 rollback
#19
Changing the status properly.
If you've suggestions, please let me know ASAP. I will mark a string freeze shortly after the BETA6 release.
#20
Have you fixed all the buggy strings in the above patch? Why have you changed to active?
#21
#22
Seems like I need to review all again... :-( or are you going to commit more other patches still missing?
#23
Please do not commit big patches... I provide a patch asap.
#24
New patch attached.
#25
The first change from using "when" to using "if" doesn't correct anything, but it's just a style preference. I personally prefer "you can change the priority when you edit the taxonomy term", in example; it would be better to let who speaks English as first language decide about what is the best way of express that thought.
The other strings proposed to change are the strings that appear in the update log, which list the SQL queries made from the updates; that is the reason I used strings that would appear similar to SQL queries, even if it's evident that they are not SQL queries.
Those strings could also be removed; I used them just because I think an update function should leave a track of what it did, even if it didn't execute any SQL queries. As that is my personal preference, I would let earnie decide about that.
#26
To add a thought more, I think that "you can change the priority if you edit the taxonomy term" has a different meaning than "you can change the priority when you edit the taxonomy term" (I know the sentence is not exactly that, but that doesn't change what I am going to say).
The proposed change has the same effect of changing "you and me" with "me and you"; who uses "you and me" could put the accent on "you", but in general "you and me" is seen equivalent to "me and you" (without a particular context associated with the nominal phrase).
#27
How about:
The default priority may be changed while adding or editing a vocabulary and the default priority can be overridden while adding or editing terms.
EDIT: Or instead of ``changed'' how about ``assigned''?
#28
"The default priority may be changed while adding or editing a vocabulary and the default priority can be overridden while adding or editing terms."
"changed" is better than "assigned". The default priority is already assigned, and "changed" describes better what the user is doing; "assigned" would mean there isn't a default value used.
#29
I like
The term default priority may be changed while adding or editing vocabularies and the term default priority can be overridden while adding or editing terms.better. The added "term" adds clarity to the sentence. Modifying "a vocabulary" to vocabularies uses the same plurality found later in the sentence with "terms".
#30
That is even better.
#31
Committed to CVS.
I've set this back to active for a later review.
#32
@earnie: I found a few more strings that should be changed... patch attached.
#33
I've committed the #32 patch. Leaving open for further string review.
#34
I am considering this issue fixed and will open a string freeze issue.
#35
There has been a bug introduced somewhere:
The sitemap is located at <a href=\\\"@sitemap\\\">@sitemap</a>.have to many backslashes.#36
If that is the full string, it would be enough to not use the
"delimiters for the string, to avoid the need of escaping the same character inside the string.#37
Yes, the above is the extracted string. Escaped double quotes in PO files normally show
\"in PoEdit.#38
The bug only exists in the xmlsitemap.pot and xmlsitemap.de.po files and not the original text. But I modified the text to remove the unneeded \.
#39
Change committed.
#40
Automatically closed -- issue fixed for 2 weeks with no activity.