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.
update_prepare_d7_bootstrap() docs say
No access check has been performed when this function is called, so no changes to the database should be made here.
but it creates tables registry, registry_file and cache_bootstrap.
Comment | File | Size | Author |
---|---|---|---|
#4 | update.inc_.patch | 729 bytes | glipay |
Comments
Comment #1
jbrown CreditAttribution: jbrown commentedComment #2
jhodgdonOdd. It seems like any function doc should not say what the function "should" do, when the function has no arguments like this one. It should just document what the function does... I think that line should just be removed.
http://api.drupal.org/api/function/update_prepare_d7_bootstrap/7
Good project for a novice doc contributor...
Comment #3
Damien Tournoud CreditAttribution: Damien Tournoud commentedI agree with #2.
To be fair, I think the intend of this comment was to say that no *irreversible* modification to the database should be made. Creating a table if it doesn't exist is perfectly safe.
Comment #4
glipay CreditAttribution: glipay commentedchanged the documentation to read "No access check has been performed when this function is called, so no irreversible changes to the database are made here."
Comment #5
jhodgdonLooks fine - thanks!
Comment #6
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD.