User tests: Successful: 3 brianteeman, sonalitailored, truptikagathara Unsuccessful: 0
Improve menu item duplicate alias error message so we don't need to guess which menu item is the one that have duplicate alias.
None.
Status | New | ⇒ | Pending |
Category | ⇒ | Language & Strings Administration Libraries |
Labels |
Added:
?
?
|
Labels |
Removed:
?
?
|
Labels |
Added:
?
?
|
I like the addition to say which menu item clashes
I dont like the rest of the novel that you wrote
the "novel" is an explanation why the error occurs so people can understand, if more people don't like i can remove that part.
You tried to call x the same as x
That's enough. Now you are naming it there is even less reason to write a
novel
ok @brianteeman you convinced me, removed the "novel".
Thanks and sorry but now I realise it does need the trashed comment still.
I made a suggestion inline
On 7 September 2016 at 19:31, andrepereiradasilva notifications@github.com
wrote:
ok @brianteeman https://github.com/brianteeman you convinced me,
removed the "novel".—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11972 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8Yz6e7qSqzSExOyC3pRLJa9rqDeoks5qnwMRgaJpZM4J3B9i
.
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
I have tested this item
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
Labels |
Added:
?
|
I have tested this item
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-09-08 06:11:01 |
Closed_By | ⇒ | rdeutz | |
Labels |
Added:
?
|
I suggest to modify this PR by modifying the string
+JLIB_DATABASE_ERROR_MENU_UNIQUE_ALIAS="The alias <strong>%s</strong> is already being used by <strong>%s</strong> menu item in the <strong>%s</strong> menu (remember it may be a trashed item)."
to
+JLIB_DATABASE_ERROR_MENU_UNIQUE_ALIAS="The alias <strong>%1$s</strong> is already being used by <strong>%2$s</strong> menu item in the <strong>%3$s</strong> menu (remember it may be a trashed item)."
as we have 3 variables that could be confused by TTs in some languages.
Labels |
Removed:
?
|
I have tested this item✅ successfully on
Based on code review
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11972.