location module declares this constant also.
Should it be renamed (to PLAYLIST_PATH)?

Comments

zirafa’s picture

zirafa commented

ha, yes. thought i had fixed this already...

zirafa’s picture

zirafa commented
Status: Active » Fixed

fixed.

yongrokc’s picture

yongrokc commented

LOCATION_PATH is also defined in location.inc...

zirafa’s picture

zirafa commented

Yes, I know that, so this should already be fixed now. I changed it to PLAYLIST_PATH.

yongrokc’s picture

yongrokc commented

Sorry, I meant LOCATION_PATH is also defined in playlist.inc...
It looks like you only committed playlist.module.

zirafa’s picture

zirafa commented

ah...whoops! i got rid of it in playlist.inc, since i don't think i needed to declare it twice.

zirafa’s picture

zirafa commented

ok! should be fixed now... (hopefully) :)

matt@antinomia’s picture

matt@antinomia commented
Version: master » 4.6.x-1.x-dev
Status: Fixed » Active

zirafa, could you roll this into the 4.6 version as well? Is it appropriate for me to reactivate as a 4.6 bug (as I am doing this moment)?

Thanks!

zirafa’s picture

zirafa commented
Status: Active » Closed (fixed)

This should be fixed in 4.6 and 4.7. I highly recommend looking into the 4.7 version as the module has been completely rewritten.