Any plan to port bawstats to D7

Comments

egfrith’s picture

Yes, 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.

egfrith’s picture

Assigned: Unassigned » egfrith

I'm aiming to start work on this very soon - I have a D7 site I need stats for!

egfrith’s picture

Version: 6.x-1.1 » 6.x-1.2
Priority: Normal » Major

Now 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.

egfrith’s picture

#1447256: D7 Release has been marked as a duplicate of this bug. I'm still very aware that the port needs to get done.

joel_guesclin’s picture

I 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?

egfrith’s picture

I 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.

egfrith’s picture

The 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.

egfrith’s picture

Fixing #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.

egfrith’s picture

Version: 6.x-1.2 » 7.x-1.0-alpha2
egfrith’s picture

I'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.

egfrith’s picture

There 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)

egfrith’s picture

I'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).

egfrith’s picture

Version: 7.x-1.0-alpha2 » 7.x-1.0-alpha4
egfrith’s picture

Version: 7.x-1.0-alpha4 » 7.x-1.0-beta1
Issue summary: View changes

If 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

  • egfrith committed 61ef9b5 on 6.x-1.x
    Issue #1104658, by egfrith: Port to D7; Get statistics to display
    
    
  • egfrith committed 8415b53 on 6.x-1.x
    Issue #1104658, by egfrith: Make the admin menu visible
    
    
  • egfrith committed bb32cf4 on 6.x-1.x
    Issue #1104658, by egrith: Fix complaints about undefined offsets
    
    
  • egfrith committed de5b09e on 6.x-1.x
    Issue #1104658, by egfrith: Removed the MimeHashLib notice
    
    
  • egfrith committed ead7b5b on 6.x-1.x
    Issue #1104658, by egfrith: Converted code to D7 format using...
egfrith’s picture

Status: Active » Fixed
egfrith’s picture

Marking as fixed. There's definitely no development work happening on the D7 version of Bawstats

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.