phpvms / phpvms_v2

Virtual Airline Management (not maintained)
http://www.phpvms.net
BSD 3-Clause "New" or "Revised" License
41 stars 46 forks source link

Cannot Edit or Delete Airport #64

Closed MikeP1990 closed 12 years ago

MikeP1990 commented 12 years ago

I tried to delete or edit the default KJFK airport but it wouldn't let me. I ended up having to change the ICAO code via the database and then edit the airport in phpVMS and do a ICAO check to gather the other information, at that point, it worked...

nabeelio commented 12 years ago

Can you look in the firefox/chrome error console to see if any errors show up?

MikeP1990 commented 12 years ago

I don't see any errors in the error console or on-page.

nabeelio commented 12 years ago

Oh, if you want to change the ICAO, then add a new airport (because technically it's a new airport). That's as designed. You said you could still edit the airport after you changed the ICAO in the database.

You can't delete airports through the interface, because there might be other data attached to it. If you've just added it, then it's safe to do it through the database.

MikeP1990 commented 12 years ago

You need to add further functionality to the code base then as not everyone wants or needs kJFK in the Airports. There are ways of forcing the user not to delete if it's associated with other aspects of the system.

Anyway, I thought I'd just bring it to your attention.

nabeelio commented 12 years ago

It's just added as an example in there. Airports will only show up in schedules if they're being used, so having it there has no effect on any schedules, etc. It's just an extra airport.

I know there are ways to do it, but it's not a priority at this point, no one's really had an issue with it.

MikeP1990 commented 12 years ago

I understand. I just thought it was an issue with the software, not allowing me to edit the Airport or remove it. Never used phpVMS and I'm just starting out into this side of the industry so... :).

nabeelio commented 12 years ago

No problem! You're right, it can be a little more clear than it is.