It's an honor to post the first issue for this module. The timing of this module release could not have been better for me. I've just started a mapping project for which this module will be perfect!

I look forward to talking with you more about the development roadmap for this module. Is views and location integration on your horizon? So that you could create an xml file that pulls nodes with location data?

Here's a quick patch I wrote that picks up the actual xml file directory instead of using a hardcoded value.

CommentFileSizeAuthor
diymap.patch558 bytesseanberto

Comments

seanberto’s picture

seanberto commented
Assigned: Unassigned » seanberto
Status: Active » Reviewed & tested by the community
seanberto’s picture

seanberto commented

heh. Just looked at your cvs commits and saw that you patched this as well. However, I don't understand why you didn't use your own get_path function defined right above where this path is created. Are you removing your module's get_path function?

If not, maybe my patch still applies?

-s

yraber’s picture

yraber commented

Views and location integration are on my horizon, but not my first priority. This is my first module and I'll have to catch up some understand before doing it :) But I'm currently rewriting all the map configuration form to use AHAH_forms, which make is much more user-friendly and I will use this opportunity to add field validation and test a little bit more everything.

Any help would of course be welcome for views/location integration.
Don't contact to contact me if you have plans/ideas.

yraber’s picture

yraber commented

Sorry, I meant : Don't hesitate to contact me.

yraber’s picture

yraber commented
Status: Reviewed & tested by the community » Fixed

This was already fixed.

yraber’s picture

yraber commented
Status: Fixed » Closed (fixed)