do not delete sites that have entities

[Nathan White]

Hi all -

Someone recently ran across a problem related to "orphaned" entities. Basically, a custom module was grabbing all the events throughout Reason, and then building URLs to these events in this manner:

1 .Grab upcoming events in Reason instance wide.

2. For each event, grab the owner site entity.

3. Build the url to an event page on the owner site.

The event_helper.php class was being used in this process, and was crashing, as it called methods on the site entity without making sure the event had a valid owner site entity. The orphaned events, of course, did not.

I have updated the code in event_helper.php to fix the crash and updating to the nightly will fix this bug, but it brings up a good point to mention:

Do not delete sites unless you have first moved or deleted all the entities on that site.

The site deletion interface will not stop you from deleting a site and orphaning entities. I hope to make some changes to enforce this in the administrative interface, but I am not going to get a chance to address this in the short term.

Nate