If I go to http://www.example.com/admin/settings/geshifilter/filterconflicts to see a list of the filter conflicts, the links to resolve these conflicts are wrong, both in the Input format and possible solutions column.

The fix is to remove the leading '/' character in the calls to l() in geshifilter_admin_filter_conflicts() (lines 711 and 714).

In other words,

-              l(t($input_format->name), "/admin/settings/filters/$format"),
+              l(t($input_format->name), "admin/settings/filters/$format"),
               $cfilter->name,
               t($conflict_info['description'], array('%cfilter' => $cfilter->name, '%geshifilter' => $geshifilter->name)),
-              l(t('Rearrange filters'), "/admin/settings/filters/$format/order", array(), drupal_get_destination()),
+              l(t('Rearrange filters'), "admin/settings/filters/$format/order", array(), drupal_get_destination()),

I'll try to post a real patch sometime later today, but it's a simple fix to get the links working right.

CommentFileSizeAuthor
#1 geshifilter_url_fix_0.txt1.02 KBaclight

Comments

aclight’s picture

Assigned: Unassigned » aclight
Status: Active » Needs review
StatusFileSize
new1.02 KB

But really, why wait :)

soxofaan’s picture

Assigned: aclight » soxofaan
Status: Needs review » Fixed

Good catch,
but you missed two other links ;)
anyway, solved in http://drupal.org/cvs?commit=83610

thanks for testing

Anonymous’s picture

Status: Fixed » Closed (fixed)