Pattern at line #120 catches and redirects any request containing taxonomy/term/* instead of handling only requests fully matching taxonomy/term/*.
This may break CKEditor Link and other modules that handle paths like ckeditor_link/revert/taxonomy/term/10

CommentFileSizeAuthor
#2 fix_taxonomy_term_pattern-1908466-2.patch840 bytesanrikun
#1 fix_taxonomy_term_pattern-1908466-1.patch832 bytesanrikun

Comments

anrikun’s picture

anrikun commented
Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new fix_taxonomy_term_pattern-1908466-1.patch832 bytes

Here is a fix.

anrikun’s picture

anrikun commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
StatusFileSize
new fix_taxonomy_term_pattern-1908466-2.patch840 bytes

This same bug is present in D7 as well.

anrikun’s picture

anrikun commented
Issue summary: View changes

Added a reference to CKEditor Link.

fangel’s picture

fangel commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

The D7 patch works as intended.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Bump. +1.

BarisW’s picture

BarisW commented

Makes a lot of sense. Thanks!

  • BarisW committed 9c099e1 on 7.x-1.x authored by anrikun 
    Issue #1908466 by anrikun, fangel, DamienMcKenna, BarisW: Wrong taxonomy...
BarisW’s picture

BarisW commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x-dev

Status: Fixed » Closed (fixed)

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