[Fixed] Batch removal of planes from leasing fails

[Chavaquiah]

Removing planes from the leasing market does not work unless it's done one by one. Selecting a few at a time and using the "Remove from lease" button results in a SQL error:

Code: [Select]

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /var/www/vhosts/airlinemogul.com/httpdocs/airlinemogul/aircraft_market.php on line 691
This on top of a previously reported error where lease conditions cannot be changes:

Code: [Select]

Putting plane up for public lease.
Unknown column 'lease_to' in 'field list'
There was a problem when updating the lease terms.

Happening in World 14, Airline 8585.

[Air Elbonia]

I think i've fixed the latter, the former... should at a bare minimum provide a slightly more descriptive error message (which will help me fix it properly).

[Chavaquiah]

OK, there is some more information now. Also notice how, on the second step, a/c types are not displayed.










Modifying lease conditions is already fixed.


Oh, BTW, there seems to have been a regression on the left navigation bar. Clicking both DOC and Airline Value take us to the finance screen instead ot view routes and rankings, as they used to these last few days.

[StephenM]

Will made a change to that file, it may have been an older version that he changed.

[Chavaquiah]

Hint: Version Control System. 

[Air Elbonia]

we're verifying a version control system install..

anyways, stephen, that's the most recent version (all i did was make it display the detail of the error).

I'll see what i can do to fix this, if stephen hasn't, after work today.

[Chavaquiah]

anyways, stephen, that's the most recent version (all i did was make it display the detail of the error).

Sorry for the confusion but my last comment was about the "Quick stat" panel (which I assume is an include). Somewhat unrelated to the mass removal bug.

[MrOrange]

anyways, stephen, that's the most recent version (all i did was make it display the detail of the error).

Sorry for the confusion but my last comment was about the "Quick stat" panel (which I assume is an include). Somewhat unrelated to the mass removal bug.

Yeah, I noticed (and re-updated) that one too after some user requests. See, we do listen to user input

Now, Will, get that Version Control System going

[StephenM]

Now, Will, get that Version Control System going

Its work in progress, slow progress but there is one outstanding issue for the moment.

[StephenM]

Fixed, simple issue of having an extra pair of ()'s!

[Chavaquiah]

Time to start pushing for a mass lease option?

Anyway, thanks. This makes using the aircraft market much easier.

There seems to be still a small glitch (see PM) but otherwise it's working fine.

Again, thanks.

[StephenM]

There seems to be still a small glitch (see PM) but otherwise it's working fine.

Replied to.

[Chavaquiah]

And fixed the last small problem, I see. It seems to be working just fine now.