For some reason, my other cleanup missed the .install file. (But then, that was mainly targetted at getting ready to work on Transliteration integration, which doesn't have much to do with the .install file, I think.)

But looking at the .install file, I do get the same itch to impose some standardisation on the code and other small stuff. So here we go.

The drupal_uninstall_schema() should possibly go into its own issue. Let me know I'll make it so.

CommentFileSizeAuthor
#2 pathauto.install_minor_cleanup.d5.patch3.92 KBFreso
#1 pathauto.install_minor_cleanup.d6.patch3.3 KBFreso
pathauto.install_minor_cleanup.d6.patch3.42 KBFreso

Comments

Freso’s picture

Freso commented
StatusFileSize
new pathauto.install_minor_cleanup.d6.patch3.3 KB

Actually, since there is no pathauto_schema() anywhere, the drupal_install_schema() should probably just be removed...

Freso’s picture

Freso commented
StatusFileSize
new pathauto.install_minor_cleanup.d5.patch3.92 KB

And here's some cleanup for 5.x.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Fixed

Committed to both 5.x and 6.x. Thanks, Freso.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.