Posted by Susurrus on June 4, 2008 at 9:49pm
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | trigger.module |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
actions_syncronize() didn't use db_placeholders() as it should've plus a few other small changes.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| actions.inc_.patch | 2.2 KB | Ignored: Check issue status. | None | None |
Comments
#1
Also, I pegged #246522: Typo in function documentation in actions.inc into this patch as it's such a small change that I figured it'd get in easier if it was part of a more worthwhile patch.
#2
All SimpleTests still pass, though I'm not sure if any test this function.
#3
Nice job. Re-rolled patch for changes to actions.inc and works perfectly.
#4
Patch still applies cleanly, passes all tests, and provides a small cleanup in code that reduces 11 lines of code to 3 without increasing complexity. Ready to be committed.
#5
I've asked John to take a look at this.
#6
#7
Tested and works as expected.
#8
Thanks, committed. :)
#9
Automatically closed -- issue fixed for two weeks with no activity.