Hi,

could you please add some more variables to the web task then just [site_url]?

This would allow to use Drupals menu hook.
E.g.: [site_url]/my_custom_hook/[process_id]/[queue_id]/

CommentFileSizeAuthor
improved_webtask_patch.diff703 bytesjantimon

Comments

_randy’s picture

_randy commented

Thanks for this Merco.

However I think this patch will break sites that do not use clean urls.

The original method will concatenate the queueid to the url. That url could then be rewritten by a rewrite rule if need be.
However I do see value in perhaps tweaking this slightly:

1. provide a radio button on the manual web task that allows one to select a custom URL or a Maestro generated URL.

2. provide 3 replacement tokens of [site_url], [process_id] and [queue_id] so that you can formulate the url as you choose when you choose custom URL.

3. if Maestro generated URL is chosen, then the default ?queueid=xxx version will be used.

Thoughts?

jantimon’s picture

jantimon commented

I totally agree with your point.

What about using a checkbox instead of radio buttons?
The title could be "Add queue id to query string".

To prevent any conflicts with previous versions it should be checked by default.

jantimon’s picture

jantimon commented

Are you still working on this issue?