I saw a .info file was added to CVS. Here's a patch for kml.module that makes that Drupal 5 compat also.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | kml_47_locationupgrade.patch | 13.71 KB | geodaniel |
| #7 | kml_5_0_0.patch | 20.43 KB | geodaniel |
| #5 | kml_5_0.patch | 14.3 KB | geodaniel |
| #4 | kml_5.patch | 11.49 KB | geodaniel |
| kml.module_patch5_0 | 2.93 KB | AjK |
Comments
Comment #1
AjK commentedComment #2
geodaniel commentedThanks AjK! It looks like there were some things in there that shouldn't have been in the 4.7 code anyway, so I'll fix those in there and then release a 5.x version too.
Comment #3
hickory commentedCould this patch be committed and a DRUPAL-5 branch made?
The only thing I'd change about the patch is to leave kml_admin_settings named the way it was. Apart from that it's quite straightforward.
Comment #4
geodaniel commentedI've spent some time on this today working from AjK's patch. There were some other bits that needed fixing as well though, mostly due to changes in the location module (like adding more than one location, and one of the location table fields changing its name). I've still got some more work to do on this, but attached is the current patch for reference.
Comment #5
geodaniel commentedSome more updates to the patch... still not ready:
Comment #6
geodaniel commentedA modified patch has been commited, the code branched for Drupal 5.0 and a release made. The last two issues from my previous comment have not yet been fixed but the module works now with 5.0, so they deserve their own issues.
I'd be very grateful if anyone interested in using the module on 5.0 could test it out and create new tickets for any issues they find.
Comment #7
geodaniel commentedOh, and I need to backport a bunch of these things back to the 4.7 branch, so attaching patch for reference.
Comment #8
geodaniel commentedBackported to 4.7 branch too - KML module should now work happily with location module again.
Comment #9
(not verified) commented