Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: Souvent22 commentedB it is.
Comment #4
m3avrck CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: Dries commentedPatch no longer applies, but needed.
Comment #14
Jaza CreditAttribution: 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 CreditAttribution: Jaza commentedComment #16
m3avrck CreditAttribution: 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 CreditAttribution: Jaza commentedOops.. thought I removed that duplicate line! New patch has this cleaned up.
Comment #18
Richard Archer CreditAttribution: Richard Archer commented+1 This patch works for me.
Comment #19
Dries CreditAttribution: Dries commentedWhen you submit a new term, you should end up on the term overview page.
Comment #20
m3avrck CreditAttribution: 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 CreditAttribution: m3avrck commentedRight, that seems to be the general opinion, in which case this patch still applies :-)
Comment #23
Jaza CreditAttribution: 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 CreditAttribution: Dries commentedCommitted to HEAD. Thanks.
Comment #25
(not verified) CreditAttribution: commented