I have several suggestions for improving (in my eyes) the user interface:

I'd like to have the settings as a tab instead of a submenu (it is better to handle especially with the DHMTL menu module as it by default doesn't display menu items that have sub-items). Also the settings are kind of hidden under a menu entry title 'Revision deletion'. The menu title (better a tab title!) should be renamed to 'Settings'

In the settings there is a string 'Age in days of revisions that should be deleted.' which I would change into 'Age of revisions that should be deleted.' as there are not only option in days but also in weeks.

There is also an untranslatable string 'No Node Revisions to delete at this time. ' on ?q=admin/content/revision_deletion.

Finally there is a string 'Select node types for revision deletion. Multiple select enabled.' which ends with a dot although there is a hardcoded ':' after it.

Thanks for the module anyway!

CommentFileSizeAuthor
#6 hardcopy.PNG17.24 KBFrankT

Comments

nancydru’s picture

Status: Active » Patch (to be ported)

Please check the patch in #547650: Coding standards. I believe it takes care of all your concerns, but still needs to be ported to 6.x.

nancydru’s picture

Status: Patch (to be ported) » Fixed
FrankT’s picture

Status: Fixed » Active

Thanks so far. After updating to the latest version there is still a menu entry under 'Revisions to Mass Delete' (/admin/content/revision_deletion), named 'Revision Deletion' which points to the settings tab (/admin/content/revision_deletion/settings).

The menu entry 'Revision Deletion can be removed, or is it a bug in my installation?

I also suggest to replace the setting 'Show conditional labels' by 'Show additional labels', so far I do not understand why the labels should be conditional.

nancydru’s picture

Status: Active » Postponed (maintainer needs more info)

This is what you should see

  $items['admin/content/revision_deletion'] = array(
    'title' => 'Revisions to Mass Delete',
    'page callback' => 'revision_deletion_settings_page',
    'access arguments' => array(REV_DEL_PERM),
    'type' => MENU_NORMAL_ITEM,
    'description' => 'Configure or manually run the revision deletion module',
    );

  $items['admin/content/revision_deletion/list'] = array(
    'title' => 'List',
    'page callback' => 'revision_deletion_page',
    'access arguments' => array(REV_DEL_PERM),
    'type' => MENU_DEFAULT_LOCAL_TASK,
    );

  $items['admin/content/revision_deletion/settings'] = array(
    'title' => 'Settings',
    'page callback' => 'drupal_get_form',
    'page arguments' => array('revision_deletion_settings'),
    'file' => 'revision_deletion.admin.inc',
    'access arguments' => array('administer site configuration'),
    'type' => MENU_LOCAL_TASK,
    'description' => 'Configure settings for the revision deletion module',
    );

I assume you did run update.php, which is necessary with the menu changes.

I called them "conditional" because they depend on the other settings as to whether or not they are produced. But I went ahead and changed them. BTW, have you looked at translation and whether there are any fixes needed for that?

FrankT’s picture

Thanks, I always run update.php after any updates, but the menu system sometimes behaves strange sometimes. I think I will uninstall the module once and reinstall it again.

I will have a look at the changes this weekend (they are not yet in the dev version of Aug 22!?), I will update my translation and post it shortly then.

FrankT’s picture

Status: Postponed (maintainer needs more info) » Active
StatusFileSize
new17.24 KB

There is just another thing: there are different date/time formats in the revisions listing, see attached hardcopy.

FrankT’s picture

I reinstalled the module, after that there is no more menu entry under 'Revisions to mass delete', so that's solved.

nancydru’s picture

Title: suggestions for the user interface » Different date formats

Changing the title to reflect the new point. I figured you had run update, but just wanted a little reminder because even I forget once in a while.

nancydru’s picture

Okay, changed the format.

nancydru’s picture

Status: Active » Fixed
nancydru’s picture

Status: Fixed » Closed (fixed)