1. In my opinion the menu entry "Automated aliases settings" should be "URL alias settings" or even better "Settings". There is an old issue about this http://drupal.org/node/322378.
  2. In the settings page for each menu entry it says "... settings" (Taxonomy term path settings, Forum path settings, User path settings etc).
    It would make sense to skip the "... settings" and just write (Taxonomy term path, Forum path, User path etc).
    The page gets cleaner and it also makes it much easier to translate!
  3. A minor bug in pathauto.inc:551
    "t('right parenthesis )')"
    should be
    t('Right parenthesis )')
  4. Another minor bug in pathauto_taxonomy.inc:146
    t('Bulk generate forum paths')
    Guess it should be
    t('Bulk generate aliases for forum paths that are not aliased')

I created a patch with these changes if you want to adopt them.
Cheers
/Magnus

CommentFileSizeAuthor
pathauto.patch6.91 KBmagnus

Comments

magnus’s picture

magnus commented
Status: Needs review » Reviewed & tested by the community
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Needs review

Usually frowned upon to RTBC your own patch.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Needs work

Let's remove change #1 for the menu path to #322378: Revise and cleanup Pathauto menu structure.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Fixed

Committed a modified version of the patch to 7.x-1.x, and backported the fix to the 'Right parenthesis' text in 6.x-1.x.
http://drupal.org/cvs?commit=336320
http://drupal.org/cvs?commit=336324

Follow-up in #713632: Split settings UI into 'Patterns' and 'Settings' pages for the rest.

Status: Fixed » Closed (fixed)

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