I'm writing a module that needs to have access to the message to be evaluated before the Token module operates on it. Unfortunately that's currently impossible because the weight of input evaluators is only respected for the _prepare hook but not for _apply. The attached patch fixes this.

CommentFileSizeAuthor
#4 rules_input_evaluators_sort.patch897 bytesicecreamyou
#2 rules_input_evaluators_sort.patch897 bytesicecreamyou
rules_input_evaluators_sort.patch926 bytesicecreamyou

Comments

Status: Needs review » Needs work

The last submitted patch, rules_input_evaluators_sort.patch, failed testing.

icecreamyou’s picture

icecreamyou commented
Status: Needs work » Needs review
StatusFileSize
new rules_input_evaluators_sort.patch897 bytes

Ah, crap. Windows line endings FTL

Status: Needs review » Needs work

The last submitted patch, rules_input_evaluators_sort.patch, failed testing.

icecreamyou’s picture

icecreamyou commented
Status: Needs work » Needs review
StatusFileSize
new rules_input_evaluators_sort.patch897 bytes

aaaand still running a version with an $Id$ tag somehow. /crosses fingers

Status: Needs review » Needs work

The last submitted patch, rules_input_evaluators_sort.patch, failed testing.

icecreamyou’s picture

icecreamyou commented

Okay. I give up. I don't know why it's not applying.

fago’s picture

fago
Primary language German
Location Vienna
commented

Did you roll it with -p1 ? Anyway, patch looks good to me. Any testers?

pribeh’s picture

pribeh commented

Seems to be working for me after having manually applied the suggested code in the patch.

mitchell’s picture

mitchell commented
Component: Miscellaneous » Rules Core

Updated component.

tr’s picture

tr commented
Issue summary: View changes
Status: Needs work » Closed (won't fix)

This piece of code does not exist anymore in D7/D8 Rules. And D6 has been unsupported for more than two years now.

If you continue to have this problem in D7 or D8, please open a new issue with details of how to reproduce the problem.