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've looked over the port again. Here's the latest attempt against 5.x-1.1. I'm including a module version and patches as well, for quick review. I'm not sure what I'm missing, but no blocks show up anywhere :( I've looked over the changes for Drupal functions, and I think I got them. Somehow I'm missing something. The table seems to install fine. I'm not sure. Could you give it a quick look?
It is gotta be close, not sure why I'm not seeing any results...
Regards,
Marius
Comment | File | Size | Author |
---|---|---|---|
#3 | user_visits_adv-6.x-1.x.zip | 11.92 KB | mariusooms |
user_visits_adv_6_port.tar_.gz | 8.36 KB | mariusooms | |
user_visits_adv_6_patches.tar_.gz | 1.71 KB | mariusooms |
Comments
Comment #1
mariusooms CreditAttribution: mariusooms commentedThe block 'Latest visitors' works fine. However the most important one, namely, the history block is a no show. The data in the table gets populated with the correct values. Somehow the theme function must be incorrect, or somewhere else the history is not being fetched properly.
I really don't know at this point.
Regards,
Marius
Comment #2
mariusooms CreditAttribution: mariusooms commentedAhhh...I think the module drupal 6 version from User Visits is slightly different., so a straight port for user_vistits_adv won't work? So the query must probably be adjusted. Could use some help verifying these findings.
Regards,
Marius
Comment #3
mariusooms CreditAttribution: mariusooms commentedSmall type in the theme function was to blame...sheesss...Drupal 6 port ready. Put through coder to clean up code as well.
Please commit this version to the 6 branch.
Thanks,
Marius
Comment #4
mariusooms CreditAttribution: mariusooms commentedComment #5
toemaz CreditAttribution: toemaz commentedCommitted on the 6 branch.
Thanks for the good work Marius.