Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Plz fix this.
Comment | File | Size | Author |
---|---|---|---|
#10 | taxonomy.module.patch_1.txt | 613 bytes | doq |
#7 | taxonomy.module_24.patch | 0 bytes | doq |
taxonomy.module.patch_0.txt | 607 bytes | doq |
Comments
Comment #1
doq CreditAttribution: doq commentedComment #2
doq CreditAttribution: doq commentedComment #3
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedplease don't set to rtbc before a review.
Comment #4
drummWe currently don't rewrite anything other than node queries. Adding taxonomy will terms many more changes than one query.
Comment #5
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedDrumm, that's not correct, we do rewriting for much more than node queries.
Comment #6
drummDoesn't work.
user warning: Unknown table 't' in field list query: SELECT t.name FROM term_data WHERE t.vid = 2 AND LOWER(t.name) LIKE LOWER('%f%') LIMIT 0, 10 in /home/drumm/drupal/drupal/C-HEAD/includes/database.mysql.inc on line 120.
Comment #7
doq CreditAttribution: doq commentedmy mistake:(.
& now?
Comment #8
Frando CreditAttribution: Frando commentedyou submitted an emtpy patch ;)
Comment #9
doq CreditAttribution: doq commentedsmth crazy with me:)
I will submit a full-tested patch in ~7 hours.
Comment #10
doq CreditAttribution: doq commentedhere you go!
Comment #11
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedI am not 100% sure, but wouldn't db_rewrite_sql need t.tid in the select statement in order to work?
Comment #12
doq CreditAttribution: doq commentedSorry, haven't understood. What do you mean?
Comment #13
killes@www.drop.org CreditAttribution: killes@www.drop.org commenteddb_rewrite_sql("SELECT t.tid, t.name FROM {term_data} t WHERE t.vid = %d AND LOWER(t.name) LIKE LOWER('%%%s%%')", 't', 'tid'), $vid, $last_string, 0, 10)
shouldn't the query look like this?
Comment #14
drummCommitted to HEAD (with the extra column suggested by killes).
Comment #15
(not verified) CreditAttribution: commented