Split from #421888: Clean og_form_add_og_audience() and make it an API function.
This will make og_form_add_og_audience() more API, as it will not be dependent on the URL.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 424174-og-determine-gids-via-form-state-7.patch | 5.02 KB | amitaibu |
| #3 | og-determine-gids-via-form-state-3.patch | 1.46 KB | amitaibu |
| #1 | og-determine-gids-via-form-state.patch | 1.37 KB | amitaibu |
Comments
Comment #1
amitaibuComment #2
moshe weitzman commentedSorry for the delay. Looks like a good idea, but needs work. Try this:
Try to create a group post by clicking on a link in the details block. For example, node/add/page?gids[]=2. Then try to uncheck group 2 so that no groups are selected. You can't - seems that 2 is always selected.
Comment #3
amitaibuPatch checks user didn't already select groups.
Comment #4
moshe weitzman commentedI am still seeing the behavior I described in #2
Comment #5
amitaibuweird, I tested it before submitting. Anyway, sorry, I'll re-check.
Comment #6
amitaibu@moshe,
I've re-checked patch from #3 and it seems to work properly:
node/add/story?gids[]=1
- Uncheck all groups
- Check all groups
- Check only other group.
Is there another scenario I'm missing?
Comment #7
amitaibuPatch with small fix to accept multiple gids + tests.
Comment #8
moshe weitzman commentedCommitted. Thanks.
Comment #9
truyenle commentedI have a situation in which try to create a group post by clicking on a link in the details block. However, the url don't have gids[] values something like node/add/page?gids[]=. Any Glue for this. I am using Drupal 6.16 and og 6.x-2.1.
Many thanks
Truyen
Comment #10
amitaibu@truyenle,
Please open another issue, as this isn't related to this (closed) issue.
Comment #11
truyenle commentedOK, thank you Amitaibu. Here it is http://drupal.org/node/804528.