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.
Hi,
When doing a coder review of the custom pagers module, I found that the new Drupal 6 Schema API is not being used in the update functions in the install file for adding new columns to the database. The attached patch addresses this issue.
Cheers,
Stella
Comment | File | Size | Author |
---|---|---|---|
custom_pagers_schema_api.patch | 2.51 KB | stella |
Comments
Comment #1
catchThese changes look good to me. Checked it applied cleanly etc. but I didn't actually run the update path. Should be good to go though.
Comment #2
eaton CreditAttribution: eaton commentedSchema API should not be used for *older* update functions that predate schema API, should they?
I'm not sure about this one, it's worth thinking about.
Comment #3
eaton CreditAttribution: eaton commentedLooks like this is indeed the right way. Checked in. Thanks!