EV-EVA / Ectroverse

Open Source Text based Space Empire Simulator
9 stars 6 forks source link

Merge fleet does not work #3

Closed EV-EVA closed 11 years ago

EV-EVA commented 11 years ago

You can split a fleet but not merge it. It is possible that this feature was never fully implemented.

jasondecarvalho commented 11 years ago

I seem to remember it not ever quite working.

EV-EVA commented 11 years ago

According to Helliens, it should have worked last time he checked. I'll check around.

bertsarens commented 11 years ago

This used to work, I dont know one whos server though.

Necrolgan commented 11 years ago

It can work... but it needs some work. Hehe =P

And yes, I had it working on my server back in the day... oh so long ago. Sadly all that work was lost.

Necrolgan commented 11 years ago

Seems you'd already done most of it... the problem this time was so simple it's embarrassing for whoever made it. =P

File: iohttp2.c Line: 5512

Change: svSendPrintf(cnt, "<input type=\"submit\" value=\"Merge\"");

Modification: Add the missing ">" value=\"Merge\""); should read value=\"Merge\">"); at the end for correct html code, it just isn't submitting the form. The function works.