The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Probably not now that we require PHP5.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 444876-remove-node-invoke-all.patch | 1.58 KB | damien tournoud |
| #4 | 444876-remove-node-invoke-all.patch | 10.21 KB | damien tournoud |
| #1 | 444876-remove-node-invoke-all.patch | 7.85 KB | damien tournoud |
Comments
Comment #1
damien tournoud commentedComment #2
dries commentedThis should go in, after the test bot did his thing.
Comment #4
damien tournoud commentedCrap. We removed the $page parameter and failed to properly update the book, trigger and upload.
Here is an updated patch.
Comment #6
damien tournoud commentedFailures in the book module are due to #444920: Book breadcrumbs are broken.
Comment #7
dries commentedThis patch only applied partially but I've committed what did apply and what did test properly.
There is more node_invoke_node() stuff to be deleted (e.g. the actual function, documentation) so I'm switching this back to 'code needs work'.
I accidentally committed this with #19837 -- sorry! I will give proper credit in the next commit.
Comment #8
damien tournoud commentedThis should take care of the remaining pieces.
Comment #9
dries commentedCommitted to CVS HEAD. Thanks DamZ. Marking this 'code needs work'. Mark 'fixed' after documentation updates.
Comment #10
damien tournoud commentedAdded entry for this in the two subpages of http://drupal.org/node/394066.