Text refers to users, not groups

foxtrotcharlie - September 10, 2008 - 15:05
Project:Smartqueues for Organic Groups
Version:5.x-1.2
Component:User interface
Category:bug report
Priority:minor
Assigned:Unassigned
Status:active
Description

On the smartqueue form under the fieldset labelled "Batch operations to perform when this form is submitted:", if I change any of the default values, for instance, un-check the "Create new NodeQueues" checkbox or select the "Delete these NodeQueues" radio button, after submitting the form and then opening it (edit) again, these selections are not saved in the form.

#1

ezra-g - September 10, 2008 - 15:06
Status:active» by design

That was an intentional design decision to force the user to specify which batch action to perform, in order to prevent accidental batch actions on large sites.

#2

ezra-g - September 10, 2008 - 15:07
Title:Values in og smartqueue configuration form not stored» Save batch creation settings values
Category:bug report» feature request
Status:by design» active

I suppose we could add a 'save this choice' type of checkbox, but I wonder if that would clutter up the design.

#3

foxtrotcharlie - September 10, 2008 - 16:55

Aah, sorry. I didn't read it properly, so didn't understand the functionality of it. On reading that whole field with less haste, I think it's clear enough, and it's good to reset the values :-). I also think that having a save this choice would clutter up the design.

One thing in this particular UI that I found confusing was the text above the checkbox "Create new NodeQueues" - "For Users of the above-specified roles who do not yet have user NodeQueues:". (see attached screenshot). I realise now that this creates new nodequeues for any existing group nodes which don't have a subqueue, but to me the text above the checkbox refers to users and not groups. Unless I'm missing something again, I can't figure out what is meant by "user NodeQueues".

AttachmentSize
2008-09-10 06-51-23 PM.png 7.66 KB

#4

ezra-g - September 22, 2008 - 14:38
Title:Save batch creation settings values» Text refers to users, not groups
Category:feature request» bug report
Status:active» fixed

Good catch, thanks. I fixed that text.

#5

Anonymous (not verified) - October 6, 2008 - 14:53
Status:fixed» closed

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

#6

moshe weitzman - November 12, 2008 - 02:31
Priority:normal» minor
Status:closed» active

FYI, that text still appears on the screenshot for this project. I zoomed in to read everything :)

 
 

Drupal is a registered trademark of Dries Buytaert.