daylightstudio/FUEL-CMS — pull request #248 by DevelopmentDocumentopia (2014-05-02 16:22:50+00:00)
Title: Related to fuel issue #246

This is what I think I have figured out, I should of never been able to create a empty location, which is what caused the strange create to update conversion I was seeing last week. This patch makes it to where you can't edit or update a empty location.

To duplicate the problem before applying this patch you can create a empty location, you can modify the empty location. And when you try to create another empty location with the same parent the create gets converted to a update of the original id by the insert_ignore.

After you apply this patch you can not longer create empty locations which solves the strange behavior.

Looking at the code it seems the intent is to never have a empty location. So I have modified is_editable_location and is_new_location to return FALSE if the location is empty.