AFAIK, the HEAD version of this module targets Drupal 4.7. Since a patch for 5.0 is in the queue, it would make sense to tag the files for DRUPAL-4-7 and create a release. Without a release, the module doesn't appear to support 4.7 in the modules listings on drupal.org. And a release would also be very helpful for people who would like to try the module but don't know how to access CVS.

Comments

rogerpfaff’s picture

rogerpfaff
he/his
Primary language German
Location Munich
commented

Take a look at http://drupal.org/node/110839

I have ported the base functionality to 4.7.5(x?)

Don't know if it's working on 5.x too

greetings
roger

David Lesieur’s picture

David Lesieur commented

taxonomy_hide already has a version that works with 4.7.x, so I guess you have missed it because it is only available through CVS... This issue aims precisely at that problem, to have a downloadable package for 4.7.x so people can find it and get it easily...

A patch has been submitted for 5.x, but I didn't try it.

orthopoint’s picture

orthopoint commented

I got this error message under 4.7.5 during set up a new group via OG:
warning: Invalid argument supplied for foreach() in /var/htdocs/www.orthopoint.de/htdocs/bilder/modules/modules/taxonomy_hide.module on line 83.
warning: Invalid argument supplied for foreach() in /var/htdocs/www.orthopoint.de/htdocs/bilder/modules/modules/taxonomy_hide.module on line 83.

So I think also the CVS version is not ok.

orthopoint’s picture

orthopoint commented

Now I use the last CVS version from laast year and I get the following error messages during set up a new group:

warning: Invalid argument supplied for foreach() in /var/htdocs/www.ot.de/htdocs/bilder/modules/modules/taxonomy_hide.module on line 83.
warning: usort(): The argument should be an array in /var/htdocs/www.ot.de/htdocs/bilder/modules/modules/taxonomy_hide.module on line 91.
warning: Invalid argument supplied for foreach() in /var/htdocs/www.xt.de/htdocs/bilder/modules/modules/taxonomy_hide.module on line 83.
warning: usort(): The argument should be an array in /var/htdocs/www.o.de/htdocs/bilder/modules/modules/taxonomy_hide.module on line 91.

David Lesieur’s picture

David Lesieur commented

@orthopoint, that's a separate issue. I just posted a patch. You might want to try it.

orthopoint’s picture

orthopoint commented

Thank you for your help. But i am not familiar with patches.

David Lesieur’s picture

David Lesieur commented

See how to apply patches. Even if this patch is not critical, it might help you with other Drupal issues to learn how to apply patches (which is quite easy). Also, it would allow you to test patches and thus help issues getting reviewed and solved. ;-)

orthopoint’s picture

orthopoint commented

Thank you. I will learn it. But I will launch my new drupal website next week. So there is no time. But after my re launch I will laern it and I will do it. What I think is, that the mistakes are not critical, only I see the error logs. I can use this module. Next week I will learn how to work with patches...

David Lesieur’s picture

David Lesieur commented
Version: master » 4.7.x-1.x-dev
Status: Active » Fixed

Just created the branch. Package will be published shortly.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)