I find that if I want to do slides with complex markup (floated effects, applying classes, &c.) for an S5 presentation, I need to use a Full HTML input format. So far, so good. However, if I want anonymous users to view these slides, I run into a problem with check_markup, in that as implemented in the s5_textfield.module, it checks the current user's access permissions against the node's selected format, and I don't make the Full HTML format available to anonymous users. Thus, anonymous users see titles for slides with "N/A" for the slide content, because check_markup is denying them access.

Comments

hackwater’s picture

Status: Active » Needs review
StatusFileSize
new1.33 KB

The check_markup's reference page says about checking input format access permissions: "Note that this will check the permissions of the current user, so you should specify $check = FALSE when viewing other people's content." Therefore, the two check_markup calls in the s5_textfield.module code ought to set the $check to false. The attached patch accomplishes this (it also removes an extra space before each call to check_markup; Quanta wanted to remove more spaces throughout the file, specifically more trailing spaces, but I wanted to keep the diff as simple as possible).

hackwater’s picture

StatusFileSize
new2.11 KB

Looks like HEAD is the Drupal 5 code; here's the patch again, diffed against Drupal 6 dev.

hackwater’s picture

Version: master » 6.x-1.1
Assigned: Unassigned » hackwater
Status: Needs review » Fixed

Bug is fixed in both dev versions (5 and 6), will do a release soon.

Status: Fixed » Closed (fixed)

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