Ticket #6887 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

[Patch] WikiCalenderMacro doesn't properly handled urls starting with '/'

Reported by: neil@dip.sun.ac.za Assigned to: anonymous
Priority: normal Component: WikiCalendarMacro
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

If an url pattern starting with '/' is used, this isn't properly stripped before the call to has_page, and so the macro never sees that such pages exist.

The attached patch fixes the bug here.

Attachments

WikiCalenderMacro_urls.diff (0.5 kB) - added by neil@dip.sun.ac.za on 03/26/10 16:01:06.
Patch to strip leading '/'s from the url

Change History

03/26/10 16:01:06 changed by neil@dip.sun.ac.za

  • attachment WikiCalenderMacro_urls.diff added.

Patch to strip leading '/'s from the url

03/29/10 06:55:32 changed by rjollos

Thanks for the patch. I will apply it to the trunk tomorrow.

03/29/10 06:55:42 changed by rjollos

  • owner changed from rjollos to anonymous.
  • status changed from new to assigned.

04/27/10 02:53:53 changed by rjollos

  • summary changed from WikiCalenderMacro doesn't properly handled urls starting with '/' to [Patch] WikiCalenderMacro doesn't properly handled urls starting with '/'.

I will go ahead and apply the patch now, but I won't have time to test. If you are able to test the latest version of the trunk, please report back if you can.

04/27/10 02:57:58 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [7892]) Properly handle URLs that start with . Thanks neil. Fixes #6887.

04/27/10 03:00:07 changed by rjollos

Also, please go ahead and list your name in the list of Contributors on the wiki page, if you like.

04/30/10 12:31:25 changed by neil@dip.sun.ac.za

Just tested trunk here - seems to be working fine.

Thanks.


Add/Change #6887 ([Patch] WikiCalenderMacro doesn't properly handled urls starting with '/')




Change Properties
Action