Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi toemaz,
I have attached here my work so far. I'm not sure if I did the install file correctly. I don't think so as I couldn't yet make the module work. Many parts do seem to work, but it is not gathering data yet.
I will look at it more, but maybe you know better than me if everything looks solid. I'm attaching it as a zip here, but I can attach patches if you prefer.
What do you think so far?
Regards,
Marius
Comment | File | Size | Author |
---|---|---|---|
#2 | user_visits_adv_6.zip | 5.29 KB | mariusooms |
#1 | user_visits_adv_6.zip | 5.3 KB | mariusooms |
user_visits_adv_6.zip | 5.48 KB | mariusooms |
Comments
Comment #1
mariusooms CreditAttribution: mariusooms commentedSorry...Included a patch in there somehow ;)
Here's a better one...
Comment #2
mariusooms CreditAttribution: mariusooms commentedAhhh....just realized I didn't implement the hook_theme correctly...please ignore the previous ones!
Comment #3
toemaz CreditAttribution: toemaz commentedI committed your last attached port and created a DRUPAL-6--1 branch.
I haven't had the time to look at it yet, but I'm releasing it as a dev snapshot anyway. The more eyeballs, the better.
Comment #4
toemaz CreditAttribution: toemaz commentedThere is an error in the code. See http://drupal.org/node/387688
Comment #5
mariusooms CreditAttribution: mariusooms commentedHi toemaz...that node leads to an access denied ;(
Comment #6
toemaz CreditAttribution: toemaz commentedSorry I wasn't aware it is for the release manager only. The error packaging error is:
Packaging error messages
* node_load(387688) failed
Comment #7
mariusooms CreditAttribution: mariusooms commentedHmmm, could this be due to a missing license file or $Id$ tag? In any case I didn't have the desired result yet and am actually curious to your input on this issue: http://drupal.org/node/388872
Regards,
Marius
Comment #8
toemaz CreditAttribution: toemaz commentedThe 6 dev snapshot is available. If any problems, post a new issue. Thanks marius!