Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#13819] - [com_associations] Do not use Copy reference to an existing target

? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
31 Jan 2017

Pull Request for Issue #13812

Summary of Changes

Prevent displaying the Copy Reference to Target toolbar button when the Target already exists

Testing Instructions

See #13812

After patch you will get:
screen shot 2017-01-31 at 11 48 22

avatar infograf768 infograf768 - open - 31 Jan 2017
avatar infograf768 infograf768 - change - 31 Jan 2017
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Jan 2017
Category ⇒ JavaScript
avatar infograf768
infograf768 - comment - 31 Jan 2017

@rdeutz @alikon

avatar alikon
alikon - comment - 31 Jan 2017

yes make sense 'to hide'

avatar alikon
alikon - comment - 31 Jan 2017

I have tested this item ✅ successfully on b3935bb

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13819.

avatar alikon alikon - test_item - 31 Jan 2017 - Tested successfully
avatar AlexRed
AlexRed - comment - 1 Feb 2017

I have tested this item ✅ successfully on b3935bb

Patch ok for me

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13819.

avatar infograf768 infograf768 - change - 1 Feb 2017
Status Pending ⇒ Ready to Commit
avatar AlexRed AlexRed - test_item - 1 Feb 2017 - Tested successfully
avatar infograf768
infograf768 - comment - 1 Feb 2017

RTC. Thanks for testing

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13819.

avatar rdeutz rdeutz - change - 1 Feb 2017
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-02-01 10:49:50
Closed_By ⇒ rdeutz
Labels Added: ? ?
avatar rdeutz rdeutz - close - 1 Feb 2017
avatar rdeutz rdeutz - merge - 1 Feb 2017

Add a Comment

Login with GitHub to post a comment