Hi there. Any plans for a Drupal 6 release? Or, if I converted it for you, would you like the help/patch?

Comments

Flying Drupalist’s picture

I would be very happy to see this module ported as well!

pkej’s picture

jcfiala, why don't you make such a patch and post it in this thread? I'm sure the maintainer and others would be happy for the help.

chasz’s picture

+1

electricmonk’s picture

Feel free to provide the patch. This is a very simple module, I'm sure it won't be too difficult. I simply don't have the time to do this at the moment.

ManyNancy’s picture

+1 as well.

jcfiala’s picture

Making a patch would be cool and useful, but at the moment I'm a little overwhelmed with paying work and real life issues, as well as my own modules. If I get a chance I'll see if I can spin something up, but it may take a few weeks.

tracerhand’s picture

I would also like to see something like this for Drupal 6.

What I envision is for users to be able to create "sets" of content, by adding tags to nodes (tags that only they can see).

dodorama’s picture

StatusFileSize
new9.47 KB

I tried to port this to drupal6.
It seems to work but please note that my php skills are really basic and the code needs review.
I used deadwood module and coder module to clean the code and implement schema api.
I hope someone will find this is useful.

dodorama’s picture

Status: Active » Needs review
electricmonk’s picture

Status: Needs review » Fixed

Seems to be ok. I'm releasing it as a development snapshot. Please comment, if no bugs are found I'll release a final version.

Thanks :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

leahmd’s picture

Version: 5.x-1.0 » 6.x-1.x-dev
Category: feature » bug
Status: Closed (fixed) » Needs review

Hello..
I have installed the D6 snapshot and the tables were created, drup_user_term etc. When navigating I get errors:

user warning: Unknown table 'user_term' in where clause query: SELECT t.* FROM drup_term_node r INNER JOIN drup_term_data t ON r.tid = t.tid INNER JOIN drup_vocabulary v ON t.vid = v.vid LEFT JOIN drup_user_term ON t.tid = user_term.tid LEFT JOIN drup_private_vocabularies ON t.vid = private_vocabularies.vid WHERE ((drup_private_vocabularies.is_private = 1 AND user_term.uid = 1) OR (private_vocabularies.is_private = 0)) AND ( r.vid = 48 )ORDER BY v.weight, t.weight, t.name in /home/scraudit/public_html/dev/modules/taxonomy/taxonomy.module on line 618. 

user warning: Unknown table 'user_term' in where clause query: SELECT t.* FROM drup_term_node r INNER JOIN drup_term_data t ON r.tid = t.tid INNER JOIN drup_vocabulary v ON t.vid = v.vid LEFT JOIN drup_user_term ON t.tid = user_term.tid LEFT JOIN drup_private_vocabularies ON t.vid = private_vocabularies.vid WHERE ((drup_private_vocabularies.is_private = 1 AND user_term.uid = 1) OR (private_vocabularies.is_private = 0)) AND ( r.vid = 5 )ORDER BY v.weight, t.weight, t.name in /home/scraudit/public_html/dev/modules/taxonomy/taxonomy.module on line 618. 

This seems to be ignoring the table prefix? I've had a look at if I could fix it myself but I'm afraid I'm too much of a noob at this point to be able to help. It would be great if it could be sorted out as this is the key to a feature I need.

Thanks

Flying Drupalist’s picture

Yep, same here. Wonderful module, please fix this bug. :D

leahmd’s picture

I've tried adding some more curly braces where it seemed they might be missing, in the .module file:

function private_taxonomy_db_rewrite_sql($query, $primary_table, $primary_field, $args) {
  global $user;
  if ($primary_field == 'tid') {
    // this query deals with taxonomy objects
    $ret = array();
    $ret['join'] = "LEFT JOIN {user_term} ON t.tid = {user_term}.tid "."LEFT JOIN {private_vocabularies} ON t.vid = {private_vocabularies}.vid";
    $ret['where'] = "({private_vocabularies}.is_private = 1 AND {user_term}.uid = $user->uid) "."OR ({private_vocabularies}.is_private = 0)";
    return $ret;
    break;
  }
}

I no longer get the missing table errors and I think the functionality is working as it should be but I'm not completely certain yet.

greenbeans’s picture

subscribing

AgentD’s picture

subscribing

tevih’s picture

subscribing... is this fixed?

QuiptimeGroup’s picture

StatusFileSize
new8.44 KB

Bug fixed module version

I have fixed all "user warning: Unknown table" - errors.

benahlquist’s picture

FYI, I ran your bug fixed module version, QuiptimeGroup; had no errors and everything seemed to work perfectly.

Thanks!
ben

trobey’s picture

Status: Needs review » Closed (fixed)