search

OSU-Net / cyder

Cyder
BSD 3-Clause "New" or "Revised" License
10 stars 12 forks source link

Don't auto add objects to containers #1022

Closed murrown closed 9 years ago

murrown commented 9 years ago

This pull request does two things.

  1. Objects are no longer automatically added to the current container when they are created.
  2. The network list on the range create form shows all networks instead of only those associated with the current container.

Note that the 1st change is general and the 2nd change is specific. For 1, I could have instead only prevented Ranges from being automatically added. For 2, I could shown the full list in all forms for any object that doesn't have an explicit ctnr or ctnr_set field. But I chose this approach because it seemed the safest and most reasonable. If necessary, I can change it to the other approach.

murrown commented 9 years ago

By the way, this should resolve https://github.com/OSU-Net/cyder/issues/1017

akeym commented 9 years ago

seems to work for me