When the module is installed it doesn't work entirely until they save the settings page. We should work on that too, but this issue is a simple solution to provide a message to the user when the module is enabled saying "Pathauto is installed. Be sure to visit the settings page (link) and save the configuration for the module to start working properly."
I imagine a drupal_set_message in the .install can handle this.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | pathauto_install_vars.patch | 2.64 KB | add1sun |
| #2 | pathauto_install_0.patch | 810 bytes | add1sun |
| #1 | pathauto_install.patch | 940 bytes | add1sun |
Comments
Comment #1
add1sun commentedGot bit by this myself. I added the message with a link to the settings page to .install for 5.x-dev from CVS. If I didn't do the patch right please let me know so I can learn. Thanks!
Comment #2
add1sun commentedCleaned up, made it an error message and added doxygen comment for the install function.
Comment #3
greggleswe talked about this in IRC a bit.
Basically - don't do whitespace changes, please do the database uppercasing and the {} (though in general that probably isn't a good idea to mix fixes on the same issue) , and add doxygen comments and us drupal_set_message('...', 'error') so that users get the idea that it's really important.
Comment #4
add1sun commentedOK, here is a new patch that goes ahead and actually installs the default variables on install. I think I got em all.
Comment #5
gregglesCommitted to 5.x--2. Thanks!
Comment #6
gregglesMeh...changing status...
Comment #7
(not verified) commented