Closed (fixed)
Project:
BAWStats
Version:
7.x-1.0-beta1
Component:
Code
Priority:
Major
Category:
Task
Assigned:
Reporter:
Created:
24 Mar 2011 at 21:45 UTC
Updated:
7 Jul 2023 at 19:04 UTC
Jump to comment: Most recent
Any plan to port bawstats to D7
Comments
Comment #1
egfrith commentedYes, it's definitely my intention to port bawstats to D7. Ideally I would get some of the issues fixed in the D6 version fixed, and then do the port. It just requires a spare evening or two, which I don't seem to have many of at the moment.
If someone else wants to provide a patch, I would be happy to test it.
Comment #2
egfrith commentedI'm aiming to start work on this very soon - I have a D7 site I need stats for!
Comment #3
egfrith commentedNow that all the bugs reported in 6.x-1.1 have been fixed and 6.x-1.2 has been released, a D7 port is my next priority. I'm unavailable of for development for the next few weeks, but once I'm back I will start in earnest.
Comment #4
egfrith commented#1447256: D7 Release has been marked as a duplicate of this bug. I'm still very aware that the port needs to get done.
Comment #5
joel_guesclin commentedI appreciate that D7 is a high priority but given that many of us are still on D6 could you have a look at this bug in 6.x-1.3?
Comment #6
egfrith commentedI have started working on the port - there is now a 7.x-1.x branch in git and I have used the coder-upgrade module to do some of the work of converting the code. However, I haven't yet got a working release. Given how many other things I have to do, I'm reluctant to make promises, but I hope to have something in the next few weeks.
@joel_guesclin: I'm prioritising the D7 port for now, but I'm happy to accept patches for #1354020: Edit config.php for language settings does not work if anyone else produces them.
Comment #7
egfrith commentedThe first alpha release for D7 should be available for download in a few minutes. It's not ready for the prime-time yet; there are some issues with PHP Notices. However, I'm hoping to fix these fairly quickly.
Comment #8
egfrith commentedFixing #1223846: Strip out online_config has removed a few warnings, though there are some left. This code is now in 7.x-1.0-alpha2.
Comment #9
egfrith commentedComment #10
egfrith commentedI've removed the Notice about MimeHashIcon. This was occuring because in the 7.x version of awstats, this variable does not appear. It was trying to be read from the mime.pm file, leading to an empty array. If this failed, it assumed 7.x format. (See #956286: Error parsing AWStats 7.x library file for more details of the format.)
I'm now getting the code to read in MimeHashLib and MimeHashFamily and check that the first element of $mime_hash_lib has 2 elements. If so, it's 7.x format. If not, it's 6.x format, and we try to load MimeHashIcon.
Comment #11
egfrith commentedThere were some errors about undefined offests on admin/reports/bawstats. This was due to this page not being redirected to admin/reports/bawstats/time/month/year as it should have been. Some tweaks to hook_menu() have fixed this. (http://drupalcode.org/project/bawstats.git/commit/bb32cf4)
Comment #12
egfrith commentedI've just realised the admin menu wasn't visble on the admin/config page. This is now fixed (http://drupalcode.org/project/bawstats.git/commit/8415b53).
I'm going to release this in the 7.x-1.0-alpha4 release (I've skipped past alpha3).
Comment #13
egfrith commentedComment #14
egfrith commentedIf the beta1 version survives in the real world, I think we may be almost there with the Drupal 7 version.
There's one existing bug that I would like to squash: #1648030: Reversion back to time section when changing date with separate sections shown
Comment #16
egfrith commentedComment #17
egfrith commentedMarking as fixed. There's definitely no development work happening on the D7 version of Bawstats