the code is not robust against deleting a type that's in use

[jospoortvliet]

Hi,

I removed a few track types - some of which were in use. This resulted in invalid track IDs and errors. Perhaps this should be handled a little more elegantly, either make it impossible or set invalid track ID's to 'unset'.

[hennevogel]

This is true for:

• difficulty_level <> event
• room <> event
• track <> event
• campaign <> target

[ChrisBr]

As discussed with @hennevogel: We will implement this by overwriting the destroy method!

[hennevogel]

Well there are also the various :dependent options for the associations. First look at those

http://guides.rubyonrails.org/association_basics.html

[ChrisBr]

@hennevogel Yeah, discovered the nice depended: restrict_with_error feature!

Event_type was also affected!

[hennevogel]

:restrict_with_error is one method of handling this situation but not neccesarrily the one that should be used for everything. This is a web app, we should help people to reach their goals. If I want to delete a difficulty_level I should be able to

[jospoortvliet]

Thanks, guys!