The html documentation in marksmarty contains hyperlinks to markdown/#overview etc. This only works for sites that have clear URLs enabled. Sites that don't have clear URLs need links like ?q=markdown/#overview. The latter also works for sites with clear URLs. I suggest changing all links to the non-clear-URL, since that works for everybody. The alternative would be to let PHP change the ULS dynamically according the system's settings, but that seems like overkill (on the other hand; it would look better).
I include a patch for the first option (against cvs revision 1.6).

CommentFileSizeAuthor
marksmarty.module.diff2.87 KBPhilippe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Philippe’s picture

Philippe CreditAttribution: Philippe commented

(I need to set the status to "Patch" if I include a patch ;-)

teradome’s picture

teradome CreditAttribution: teradome commented
Assigned: Unassigned » teradome

I needed to update the syntax documentation anyway to reflect the inclusion of PHP-Markdown v1.0. Saw those links and knew they were not kosher. I'm going to try addressing this as pure namerefs, a la href="#overview" since most user-agents will change page locations on the local copy without making a new request--which is preferable. Let's hope Drupal is cool with that.

teradome’s picture

teradome CreditAttribution: teradome commented

Drupal was not cool with it (forgot about the BASE tag the theme system throws in) so the "?q=markdown/#anchor" format was applied to CVS. Will be applied to 4.4 later tonight. I will close this issue after I get some confirmation feedback on the change.

Anonymous’s picture

(not verified) CreditAttribution: commented