Closed (fixed)
Project:
Subdomain
Version:
6.x-1.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Aug 2008 at 19:50 UTC
Updated:
15 Sep 2008 at 06:21 UTC
Jump to comment: Most recent file
Hi
I tried module with drupal 6.3. I have created taxonomy but module have just one options - Create subdomains for users. Why? :(
Please help me.
Thanks.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | subdomain.patch | 2.13 KB | akor |
Comments
Comment #1
akor commentedseems drupal have hierarchy taxonomy by default (http://drupal.org/node/196600)
Comment #2
akor commentedother code to fix subdomain generation for taxonomy
Comment #3
akor commentedand one more fix
Comment #4
setvik commentedThanks for the fixes. Any chance you can submit as a patch? That'll help me review and test it more quickly.
Comment #5
setvik commentedComment #6
akor commentedI'll try to figure out how to make it :)
Comment #7
akor commentedI hope, I did it in the right way :)
Comment #8
akor commentedComment #9
setvik commentedHi Akor,
Thanks so much for taking the time to roll those into a patch. That helps a lot!
I have one more favor - sorry! I should have asked this earlier.
Regarding #1 above: it makes sense and works great. I've applied it to my working copy and will check it in soon.
Regarding #2 and #3: Could you explain the reasons for those changes? Do they fix bugs? If so, what are the steps to reproducing the problem(s) fixed by the changes in comments #2 and #3.
Thanks in advance :-),
Stein
Comment #10
akor commentedHi,
Yes #2 and #3 fixed bugs. Problem was that drupal 6.3 have not array $object->taxonomy[$vocab] anymore. It have only $object->taxonomy like array with TIDs.
Thanks.
Comment #11
setvik commentedI simplified patch #3 and applied it.
I haven't been able to reproduce the bug you say is fixed by patch #2, but I went ahead and committed it.
If you have time, can you post steps to reproduce the bug fixed by patch #2.
I should be able to roll a new release candidate today.
Thanks again,
Stein
Comment #12
setvik commentedComment #13
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.