When I create a workflow called "Product Review & Publish" it shows up on the product page under workflow as "Product Review & Publish".

Comments

reg’s picture

reg commented

Hmm, that's meant to read shows up as "...& amp ;..." without the spaces.

reg’s picture

reg commented

Never saw a response to this. Is this module still supported?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: Text clean up » ampersand replaced with html entity on "product page"
Priority: Minor » Normal
Status: Active » Postponed (maintainer needs more info)

I tried to get this to happen again but couldn't. Perhaps you are using an older version?

What page is the "product page" ?

verta’s picture

verta commented
Version: 5.x-1.1 » 6.x-1.x-dev
Status: Postponed (maintainer needs more info) » Active

I have duplicated this in 6.x-1.x-dev.

If I put in a workflow with a title of Q&A, it will be saved with the ampersand HTML encoded as


Q&A

The workflow title can't be changed once saved, which is kind of annoying. There is SO much work to set up all the check boxes if you have even a few steps and a few roles, that if a feature request to make this editable would be entertained, I would enter it. If there already is one, I will be happy to +1 it.

verta’s picture

verta commented
Title: ampersand replaced with html entity on "product page" » workflow title needs to be not html encoding special chars

changing the title, perhaps to bump the issue?

I put an ampersand in the title of a workflow, and it is now

Q&A Workflow

and I can't remove the encoding or change the title.

Would it help to change this to a feature request to be able to edit the title?

Bastlynn’s picture

Bastlynn commented
Status: Active » Closed (won't fix)

Since this request is over a year old, I'm going to assume a solution was found or you've moved on. If not, please get updated to the latest versions of all modules and make a request for it against Drupal 7 if the problem is still there and I'll be glad to take a look at it.

verta’s picture

verta commented

I don't have a D7 instance to test against, but thanks. I know D7 has been out a while, but upgrading is not an option right now, there are modules we need that are not migrated.

Is D6 unsupported by this module going forward?

Bastlynn’s picture

Bastlynn commented
Status: Closed (won't fix) » Postponed

We are currently working on a path for 7 at the moment, but 6 isn't abandoned. If you're not on the latest 6 release for Workflow, go ahead and get to that if you can - if you're already at that point let me know and I'll take a look at this once I get some time to put to it.

Bastlynn’s picture

Bastlynn commented

It looks like theworkflow title is saved with the & into the database but on display then being sent through check_plain. (This is important to avoid javascript injection issues.) However, I'm thinking check_plain may not be helping, especially if that value is then filtered a second time. I'm going to keep looking.

reg’s picture

reg commented

Are you sure this is worth pursuing? I posted this issue in 2007, surely there must be a lot of changes to the module since then making it suspect that this is even an issue anymore, even if it was never directly addressed.

Bastlynn’s picture

Bastlynn commented

If an issue has a pulse attached to it (aka active people responding) then it gets more weight to it. It's small, but still worth looking at while I'm in the code. :)

reg’s picture

reg commented

You learn something new everyday.

johnv’s picture

johnv
Primary language Dutch
commented
Title: workflow title needs to be not html encoding special chars » workflow title needs to be not html encoding special chars (D6)
Issue summary: View changes

Cannot reproduce this in D7. Reflecting this in the title.

johnv’s picture

johnv
Primary language Dutch
commented
Status: Postponed » Closed (works as designed)

Following #10, and closing this issue.