Closed (fixed)
Project:
Drupal core
Component:
taxonomy.module
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
14 Sep 2005 at 02:08 UTC
Updated:
13 Dec 2005 at 15:20 UTC
Jump to comment: Most recent file
Adding a taxonomy term should trigger a drupal_set_message() so people know somehting happened.
| Comment | File | Size | Author |
|---|---|---|---|
| #17 | term_add_feedback_0.patch | 997 bytes | Jaza |
| #14 | term_add_feedback.patch | 884 bytes | Jaza |
| #11 | taxonomy.module_10.patch | 957 bytes | m3avrck |
| #4 | terms_0.patch | 943 bytes | m3avrck |
| #3 | terms.patch | 943 bytes | Souvent22 |
Comments
Comment #1
Souvent22 commentedDrumm,
I have it fixed/patched, but before I post, we got some options:
Current way is that after adding a term, it stays on the "add terms" page....
So...
A) After adding, take me back to add terms for another one.
B) After adding, take me back to my list of terms for that vocab.
C) Give me a check box that says "Add another after sumbission"
During SAT's, they always said B was the right answer...or was it C..hmm....ideas suggestions?
Comment #2
m3avrck commentedI would say go with B. That is the general behavior in the rest of Drupal, might as well be consistent. And the checkbox doesn't help that much... extra click (I'm assumed off by default) and the user can just click 'add' again anywho.
Comment #3
Souvent22 commentedB it is.
Comment #4
m3avrck commentedTested patch, definetly improves usability of taxonomy module and adding terms. Before, it wasn't clear if a term was added or not. This also makes adding a term more consistent with how the rest of Drupal works (e.g., taking you back to the main add term page instead of a blank form wondering what just happened).
Fixed coding style of patch. Ready to go.
Comment #5
morbus iffSomewhere along the line, I think "visual feedback when terms were added" was actually REMOVED when the free tagging patch hit core. Consider: you write a node, with free tagging enabled, and you create 10 new terms. Upon successful submission of a node, you get ten messages saying new terms were added. That may be find for an administrator, but I certainly wouldn't want a regular non-admin or non-taxonomy-admin user seeing a message that they've added a new term.
Comment #6
robertdouglass commented+1 for the message. I thought that going back to the add-term form was by design, and personally, I like it. I often have to add lots of terms in a row, to the same vocab, and this saves me a page load each time. So -1 for
drupal_goto('admin/taxonomy/' . arg(2));.Comment #7
robertdouglass commentedRE Morbus's comment - we could check to see if the user is looking at the admin/taxonomy/vid/add/term path and only show the message then.
Comment #8
morbus iff(Toggling back to "needs work" per my comment in #5 - Rob's comment and my own crossed in the stream).
Comment #9
morbus iffComment #10
Kobus commentedI am with Robert here (#6) +1 for the message, -1 for the drupal_goto. The feature of adding several terms after each other is very handy.
Kobus
Comment #11
m3avrck commentedNew patch that fixes the goto() and sends you back to add another term, displaying a message that a term has been added.
Tested with free tagging, this patch does not output any messages saying "term 1 created", "term 2 created" , etc. All that is output is "Page was created" so Morbus your issued is already addressed elsewhere.
Comment #12
drummComment #13
dries commentedPatch no longer applies, but needed.
Comment #14
Jaza commentedNew patch that restores the feedback message when a new term is added. Feedback message is not displayed when adding free-tagging terms through the node add/edit screen.
Comment #15
Jaza commentedComment #16
m3avrck commentedPatch works, however the code can be cleaned up a bit, the test
$op != t('Submit')is duplicated in this patch.Comment #17
Jaza commentedOops.. thought I removed that duplicate line! New patch has this cleaned up.
Comment #18
Richard Archer commented+1 This patch works for me.
Comment #19
dries commentedWhen you submit a new term, you should end up on the term overview page.
Comment #20
m3avrck commentedDries, there was a lot of talk about this. That was my original thought too, but too many people said they liked the fact that there on the 'add term' page again so they could input a lot of terms really fast without clicking everywhere. That is how it currently works, this patch above changed it to what you said, got a lot of flak, and it was reverted once again. I'm mixed as to what the best approach is.
Comment #21
webchickYes, please *don't* send me back to the overview page after adding a term. A lot of my vocabs have upwards of 20+ terms and if I had to click "Add term" each time that would drive be more bonkers than I already am. :P
Comment #22
m3avrck commentedRight, that seems to be the general opinion, in which case this patch still applies :-)
Comment #23
Jaza commentedAgreed: sending the user back to the overview page is a PITA. (Which is why my patch doesn't do it ;)).
Comment #24
dries commentedCommitted to HEAD. Thanks.
Comment #25
(not verified) commented