search

Civcraft / Citadel

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/Citadel
BSD 3-Clause "New" or "Revised" License
6 stars 23 forks source link

Return rein_id when creating a reinforcement #150

Closed rourke750 closed 8 years ago

rourke750 commented 8 years ago

Change insert_rein to a procedure and get the rein_id from the resultset instead of how the system is now.

ProgrammerDan commented 8 years ago

Been a while since I made a plug, but -- reason #9281 to switch to PostGreSQL: INSERT ... RETURNING; -- http://www.postgresql.org/docs/8.2/static/sql-insert.html

No comparable in Mysql/MariaDB.

The change you're scoping is best for what we have now though, A+.

ttk2 commented 8 years ago

Postgre is clearly the future but we don't have the man power to port everything so soon. On Dec 2, 2015 10:49 AM, "Daniel Boston" notifications@github.com wrote:

Been a while since I made a plug, but -- reason #9281 to switch to PostGreSQL: INSERT ... RETURNING; -- http://www.postgresql.org/docs/8.2/static/sql-insert.html

No comparable in Mysql/MariaDB.

The change you're scoping is best for what we have now though, A+.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/Citadel/issues/150#issuecomment-161361475.