Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.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 CreditAttribution: Damien Tournoud commentedComment #2
Dries CreditAttribution: Dries commentedThis should go in, after the test bot did his thing.
Comment #4
Damien Tournoud CreditAttribution: 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 CreditAttribution: Damien Tournoud commentedFailures in the book module are due to #444920: Book breadcrumbs are broken.
Comment #7
Dries CreditAttribution: 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 CreditAttribution: Damien Tournoud commentedThis should take care of the remaining pieces.
Comment #9
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks DamZ. Marking this 'code needs work'. Mark 'fixed' after documentation updates.
Comment #10
Damien Tournoud CreditAttribution: Damien Tournoud commentedAdded entry for this in the two subpages of http://drupal.org/node/394066.