Some of the default patterns are "dangerous" - http://drupal.org/node/73232 - and should be changed.

Currently all of the patterns are dangerous to one extent or another.

CommentFileSizeAuthor
#2 non_dangerous_defaults.patch3.64 KBgreggles

Comments

greggles’s picture

Assigned: Unassigned » greggles
Priority: Normal » Critical

and I consider this critical (and will fix it shortly) now that we've got the 2.x branch I can change defaults again. Yay!

greggles’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new3.64 KB

how about the following default patterns

- $settings['patterndefault'] = t('[title]');
+ $settings['patterndefault'] = t('content/[title]');

- $settings['patterndefault'] = t('[vocab]/[catpath]');
+ $settings['patterndefault'] = t('category/[vocab]/[catpath]');

- $settings['patterndefault'] = t('user/[user]');
+ $settings['patterndefault'] = t('users/[user]');

- $settings['patterndefault'] = t('blog/[user]');
+ $settings['patterndefault'] = t('blogs/[user]');

- $settings['patterndefault'] = t('user/[user]/track');
+ $settings['patterndefault'] = t('users/[user]/track');

As far as I can tell, this is RTBC. I'll let folks comment over the next few days and then commit it to the 5.2 branch.

greggles’s picture

Version: 5.x-1.x-dev » 5.x-2.x-dev
Status: Reviewed & tested by the community » Fixed

webchick +1'd the idea, so I committed it. I'll still take feedback, but at least this is a start.

Anonymous’s picture

Status: Fixed » Closed (fixed)