Port taxonomy integration to Drupal 7 [#702632]

Both of these categories are rather short and don't affect many modules, so they're being combined. Refer to http://drupal.org/node/394070 to see which changes to make.

Comment	File	Size	Author
#4	all-taxonomy.patch	25.23 KB	Island Usurper
#4
#1	702632_1.patch	8.67 KB	Andy_Lowe
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Andy_Lowe CreditAttribution: Andy_Lowe commented 8 February 2010 at 03:39

File	Size
702632_1.patch	8.67 KB

First patch.
This updates all the taxonomy issues except: All taxonomy functions relating to nodes have been removed or refactored
I don't have a good grasp of what this entails yet. Here are a few notes related to this issue:

Calls to taxonomy functions which are probably going away according to http://drupal.org/node/412518

taxonomy_node_get_terms_by_vocabulary
uc_cart.api.php line 187
uc_catalog.module lines 232 and 900

taxonomy_term_count_nodes
uc_catalog.module lines 669, 837, 879
uc_catalog.pages.inc line 79

taxonomy_select_nodes
uc_catalog.module line 684 COMMENTED OUT

taxonomy_term_count_nodes is replaced with field_attach_query and I made a first attempt at the call in uc_catalog.module on line 669. I'm almost certain I got it wrong, but I can't easily test it now, so I'm moving on for a while.

Comment #2

BenK CreditAttribution: BenK commented 8 February 2010 at 20:25

Subscribing...

Comment #3

Andy_Lowe CreditAttribution: Andy_Lowe commented 13 February 2010 at 04:27

Assigned:

Unassigned

» Andy_Lowe

Comment #4

Island Usurper CreditAttribution: Island Usurper commented 15 February 2010 at 20:06

Status:

Active

» Fixed

File	Size
all-taxonomy.patch	25.23 KB

OK, from what I can see, I think I've got the field API stuff for taxonomy figured out. I discovered that a bit of the documentation on the update page was a little out of date. Apparently the taxonomy_vocabulary_node_type and taxonomy_term_node tables don't exist any more. Fortunately, there's still taxonomy_index, but it can be turned off if someone does some funny things with the field storage of taxonomy terms.

I've also discovered that I forgot to change uc_product_kit to the new Database API. :-/