Closed (fixed)
Project:
OG Galleries
Version:
5.x-1.4
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2007 at 09:56 UTC
Updated:
22 Feb 2007 at 12:15 UTC
In the settings screen you can now opt to make the gallery selection required. I also added a title to the selection box because when it is required and you select nothing the title field is used to indicate that a selection should be made.
| Comment | File | Size | Author |
|---|---|---|---|
| og_galleries.patch | 1.6 KB | jax |
Comments
Comment #1
jax commentedComment #2
karens commentedI don't really understand this. The gallery selection must be required and cannot be optional. What exactly are you trying to accomplish??
Comment #3
jax commentedWhen creating an image the og_galleries module will display a select box where you can select the gallery to put the image in. By default this select isn't required. You can just submit the image without selecting a gallery. I want to make it required to select a gallery so that it is not possible to submit an image without selecting a gallery.
If that still isn't clear enough I'll submit screenshots to illustrate.
Comment #4
jax commentedHere are the screenshots.
This is the default "Create Image" page with og_galleries enabled. Here you see what happens when you click preview. There is only an error on "title" and nothing is mentioned about the gallery.
When you add:
The create screen looks like this. Notice the asterisk? When you click preview you get this screen. Now you get an error about not selecting a gallery.
So the patch enables the administrator to make the gallery selection required. I need this for my project that is why I made the patch. If it is not committed I will have to maintain my own branch of the code which isn't fun. Moreover my patch leaves all the defaults as-is so nobody will have trouble because of it.
Comment #5
karens commentedNow that I understand, I agree it's a nice option. Committed to all branches. Thanks!
Comment #6
(not verified) commented