User tests: Successful: Unsuccessful:
This PR is an attempt to clarify the copy/move issues users are reporting with the Batch processing and is based on an idea from Crystal
The div highlighted below is hidden until a category is selected - thanks @dgt41
Make sure that batch processing still works in all core components and menus and categories
Status | New | ⇒ | Pending |
Labels |
Added:
?
?
|
Category | ⇒ | Administration Language & Strings UI/UX |
Easy | No | ⇒ | Yes |
Never expected you to agree.
Getting an error on move or copy of any articles:
I cant repeat that
On 6 August 2015 at 00:06, jwaisner notifications@github.com wrote:
Getting an error on move or copy of any articles:
[image: screen shot 2015-08-05 at 18 06 41]
https://camo.githubusercontent.com/d21dbeeabc30c08f4a32782dd2f0493c1c154a47/687474703a2f2f6973737565732e6a6f6f6d6c612e6f72672f75706c6f6164732f312f32613964376433303663386562326162363064666463383735646335383530662e706e67
This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/7637
http://issues.joomla.org/tracker/joomla-cms/7637.—
Reply to this email directly or view it on GitHub
#7637 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
I tested again with a hard reset of all github files to clear out any possible anomalies, but I am still getting the same error when attempting to batch move articles, or copy the articles.
Ahh, I was using the Master Branch... I will need to retest tomorrow on staging.
PR works as intended. Especially when you use the right branch :)
actually test has one error noted below in categories:
Updated with new code from @dgt41
Hmm.. Okay I will attempt to rebuild my setup this weekend. Appears something isnt right.
After rebuilding my setup I can say that the PR successfully works on the Staging branch with no errors.
Milestone |
Added: |
Well done! Test was successful with articles, banners and my own component.
One thing I wonder: Is it needed to change the following strings?
The following is still in use, but could easily changed to use JLIB_HTML_BATCH_MENU_LABEL
The only one we still need is COM_MENUS_BATCH_MENU_LABEL because it has a reference to the menu in it.
That way translators don't have to translate the exact same string multiple times.
For reference: 3rd parties only need to remove the leading "COM_FOO_BATCH_TIP" to stay cinsistent with the core extensions. If they leave it there, no harm anyway.
All the rest is applied automatically.
My approach with the language strings is to always assume there was a good
reason it was done that way originally even if I could not see it. So if
there was a string per component I kept it that way. It might not make
sense in en-gb to have multiple strings but I don't know about other
languages so took the safe option. (We do this a lot throughout the
strings). It doesn't bother me if we keep or remove them.
Thanks for testing. If someone can make a decision on the strings we can either move to RTC or I can update them and move to RTC
The JLib strings were introduced with 4e6f0ee (JC http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=22590).
I think the ones in the component were replaced with this, or were never used to begin with (copy-pasting stuff).
We could remove them, but we decided to not remove strings for B/C reasons.
I would just leave them alone. With J4 we will have a lot to clean up there
It would be great if we can replace the use of COM_CATEGORIES_BATCH_CATEGORY_LABEL with JLIB_HTML_BATCH_MENU_LABEL. What do you think?
If thats what you want I will do it. Cant see the point myself but I will
do what I am told?
On 7 August 2015 at 09:17, Thomas Hunziker notifications@github.com wrote:
It would be great if we can replace the use of
COM_CATEGORIES_BATCH_CATEGORY_LABEL with JLIB_HTML_BATCH_MENU_LABEL. What
do you think?—
Reply to this email directly or view it on GitHub
#7637 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
The point would be to reduce the amount of redundant strings we have, to make life easier for translators in the end.
It doesn't matter much. Would just be a small improvement I think
+1 to create a new string "JLIB_HTML_BATCH_MENU_LABEL" as my tests show that it is translated the same in all languages.
Also, please do not forget weblinks in that case as the constant would change.
Merged. I did change the categories batch to use the string from jlib and also added deprecation comments to the unused strings.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-07 10:14:52 |
Closed_By | ⇒ | brianteeman |
thanks - are we adding deprecated comments now on strings - we have hundreds
On 7 August 2015 at 11:13, Thomas Hunziker notifications@github.com wrote:
Merged. I did change the categories batch to use the string from jlib and
also added deprecation comments to the unused strings.—
Reply to this email directly or view it on GitHub
#7637 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
Good question. I thought we occasionally did that already earlier, but now when I search for it I didn't find any.
Imho it could make sense especially when there is a replacement by a global one. Maybe not so much if it's just no longer in use.
I can revert it easily if it adds more confusion than it helps.
IIRC We have a srcipt that can detect unused language strings (but i can't remember where it was)
The script is from @adaneinspain not 100% perfect as there are some false
positives
On 7 August 2015 at 11:27, zero-24 notifications@github.com wrote:
IIRC We have a srcipt that can detect unused language strings (but i can't
remember where it was)—
Reply to this email directly or view it on GitHub
#7637 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
For reference, this script: https://github.com/Jensen-Technologies/Find-Unused-Joomla-Language-Constants
Yes thats the one. Warning if using it do it one language file at a time and double check the unused strings it reports as it wont find combination strings
I am not a fan of a dropdown with :
- Don't move or copy -
as title