Check for the git code it would be great to have a drush dl possibility.
drush dl tagadelic-7.x-1.x
would help a few people ;)
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 1160136.patch | 6.57 KB | jtwalters |
| #4 | 1160136.patch | 6.77 KB | jtwalters |
| #6 | tagadelic--1160136.patch | 10.63 KB | jtwalters |
Comments
Comment #1
clemens.tolboomComment #2
clemens.tolboomThanks.
Comment #3
jtwalters commented*EDIT: see #6*
The module didn't appear to be ported to 7.x at all.
Comment #4
jtwalters commented*EDIT: see #6*
This patch gives me tagadelic blocks without errors...
Comment #5
flokli commentedWorks for me, thanks :-)
Things fixed with this patch:
One thing left:
Acessing tagadelic/list/1,2,3 throws this fatal error:
When visiting the next page, I also see the following errors (which seem to come from the page before):
Comment #6
jtwalters commentedThis should fix the issues you listed...
Comment #7
flokli commentedfixed them, getting close ;-)
but there are some new notices/strict warnings on tagadelic/list/1,2,3 :
Comment #8
jtwalters commentedI found out that there was already a "stable" version located at https://github.com/berkes/tagadelic
I wish I had found out earlier so I didn't spend all that time fixing the broken 7.x-1.x branch. Anyway, try that one and see if it works better.
Comment #9
clemens.tolboom@jtwalters [edit: and flokli]: sigh ... you should have created a new issue as your finding are not related to this particular issue
(remember: each issue has notification)
We now have a D7 branch and can so assign these to the D7 version ...
As the maintainer said in #1157380: Move D7 code to D.o the git repo's should be equal ... I do agree with you that two git repo's is awkward ... maybe you could file a new issue about that?
I fix this again ;)