When the "Double Tree" mode is enabled, a link is visible at the top of the page, called "Disable Double Tree".
Clicking on this link should allow the user to disable the "Double Tree" mode.
The URL the link points to is however broken, because of an unnecessary extra whitepace.

E.g. (notice the whitespace between voc/ and vocabulary):

admin/structure/taxonomy_manager/voc/ vocabulary_1

The link should be:

admin/structure/taxonomy_manager/voc/vocabulary_1

CommentFileSizeAuthor
#6 taxonomy_manager-broken-disable-double-tree-link-2303443-4.patch1.25 KBdevad
#4 taxonomy_manager-broken-disable-double-tree-link-2303443-4.patch1.25 KBdevad
#1 taxonomy_manager-double-tree-disable-broken-link-2303443-1-D7.patch626 bytesstefanos.petrakis

Comments

stefanos.petrakis’s picture

stefanos.petrakis
Primary language English
Location Biel, Switzerland
commented
Status: Active » Needs review
StatusFileSize
new taxonomy_manager-double-tree-disable-broken-link-2303443-1-D7.patch626 bytes

Patched!

stefanos.petrakis’s picture

stefanos.petrakis
Primary language English
Location Biel, Switzerland
commented
Issue summary: View changes
wonder95’s picture

wonder95 commented
Status: Needs review » Reviewed & tested by the community

I ran into this same problem, and the patch in #2 fixes it.

devad’s picture

devad commented
StatusFileSize
new taxonomy_manager-broken-disable-double-tree-link-2303443-4.patch1.25 KB

Same patch from #1, uploaded again to pass testing

devad’s picture

devad commented
Status: Reviewed & tested by the community » Patch (to be ported)
devad’s picture

devad commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new taxonomy_manager-broken-disable-double-tree-link-2303443-4.patch1.25 KB

One more attempt to send patch for automated testing...

devad’s picture

devad commented

Well... it doesn't work for me... no matter what "Status:" is selected automated patch testing is not initialized.

Sorry, for overposting. Patches are new field for me... still learning. Am I doing something wrong?

Also... I have tried to keep stefanos.petrakis as author of my uploads as well, but it didn't work. Drupal.org added me as author automatically. Sorry stefanos. That was not my intent.

devad’s picture

devad commented

And, yes, by the way... patch commit would be nice (patch #1 since it has proper credits attached).

devad’s picture

devad commented
Status: Needs review » Reviewed & tested by the community

I thing I have figured out why submitted patches are not tested automaticaly here in Taxonomy manages issues.

It took me some time... so, I'm posting just to save some time to others.

https://www.drupal.org/node/2465843

biarr’s picture

biarr commented

The patch is working as expected. Thanks!

  • VladimirAus committed 36c5712 on 7.x-1.x authored by devad 
    Issue #2303443 by devad, stefanos.petrakis, wonder95, biarr, VladimirAus...
vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Assigned: stefanos.petrakis » Unassigned
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.