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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Freso’s picture

Freso CreditAttribution: Freso commented
FileSize
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 CreditAttribution: Freso commented
FileSize
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
CreditAttribution: greggles commented
Status: Needs review » Fixed

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

Anonymous’s picture

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

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