| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
| |
Same thing, but shorter.
|
|
|
|
|
|
|
|
| |
The main change here is to remove the separate calling of 'perform' on all
beings, and rather rely on the beings to do whatever they were doing in
that function when the virtual 'update' function is called.
Reviewed-by: Yohann Ferreira
|
|
|
|
|
|
|
|
|
|
|
| |
When a prepared SQL statement would have returned multiple rows, this
function would try to set the column header names multiple times which
throws the AlreadySetException.
Currently it doesn't seem that any prepared statements are meant to return
multiple rows.
Reviewed-by: Yohann Ferreira
|
|
|
|
|
|
|
|
|
|
|
| |
Without a timeout, trying to write to a locked database fails immediately.
Now it will only fail when it can't retrieve a write lock within a second.
Still, we need to go through all the SQLite code some time, since there
is a lot of error handling missing. We also need to make sure that errors
are handled correctly on the higher level.
Reviewed-by: Yohann Ferreira
|
|
|
|
|
|
|
|
| |
* Rely on the fact that a std::string is empty by default
* Use std::string::empty() rather than comparing to ""
* Construct with std::string() rather than from ""
Reviewed-by: Bertram
|
|
|
|
|
|
| |
A C++ developer should be able to recognize a constructor and a
destructor by just looking at it, so let's stop writing down the
obvious. :)
|
|
|
|
| |
Trivial fix.
|
|
|
|
|
|
|
|
|
|
| |
This permits to avoid a memleak with the former vector form
and to use the 'place' variable when binding.
The badly prepared statements are also a bit better handled now.
With this patch, IMHO, the MySQL support is in shape.
Reviewed-by: Jaxad0127.
|
|
|
|
|
|
| |
Also added an header to the autoattack.{h,cpp} files.
Big but trivial fix.
|
|
|
|
| |
Trivial fix.
|
| |
|
|
|
|
|
|
| |
No need to fiddle around with "startTransaction" booleans now that the
helper class is a little more intelligent. When the database is already
performing a transaction, no new one will be started.
|
|
|
|
|
|
| |
Avoids having to remember to call rollbackTransaction and makes
transactions exception-safe (since the destructor of PerformTransaction
will be called when an exception is thrown).
|
| |
|
|
|
|
| |
Implementing the prepared statements in mysql backend
|
|
|
|
|
|
|
|
|
| |
Some methods were documented twice. Stuff moved to source file since
this keeps the header more readable, and makes it easier to keep the
documentation up to date with the implementation. Also, my IDE jumps to
the definition when navigating.
Removed two methods that were without an implementation.
|
| |
|
|
|
|
|
|
| |
Also updated the headers to refer to the GPL by URL instead of
suggesting to contact the FSF by snail mail, as per the latest
GPL usage instructions.
|
| |
|
|
|
|
|
|
| |
Removed pointless void in method parameter lists, fixed methods and
variables that started with upper case, removed pointless 'const' for
stuff passed by value, made some getters const, etc.
|
|
|
|
|
| |
Includes a database update. Use update_6_to_7.sql to update existing
sqlite databases.
|
| |
|
| |
|
| |
|
|
|
|
| |
Also renamed Guild::totalMembers to Guild::memberCount
|
|
|
|
|
| |
Fix for mysql backend (again)
Lua modification for integer and string ask with NPC, and closing fix.
|
|
|
|
| |
mysql backend
|
| |
|
| |
|
|
|
|
|
|
| |
tmw_items table in the database.
* Added method to DataProvider to escape quotes in strings so that they can be used in SQL statements.
|
|
|
|
|
|
| |
by nested transactions.
Fixed deletion of characters which left over garbage in sqlite.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and postgresql. The create table statements have been completely removed out from the c++ source into separate, provider specific sql files. Accountserver will no longer create a sqlite file if none present.
* Added database specific config parameters to configure each provider independent.
* Simplified the connect routine of DALStorage class since every dataprovider is now responsible to retrieve its own parameters.
* Extended abstract dataprovider to support transactions, functionally implemented for SQLite and mySQL.
* Added methods to retrieve last inserted auto-increment value and the number of modified rows by the last statement.
* Rewrite of DALStorage class to be a little more transactional.
* Fixed a bug when deleting a character. Old function left data in quests table and guilds table.
* Doxygen now also includes non-documented functions and provides a dictionary for all classes
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
MovingObject and Player classes.
|
| |
|
| |
|
|
|
|
| |
reference-counted smart pointers to facilitate the memory management.
|
| |
|
| |
|
| |
|
|
|
|
| |
class (to be completed).
|
| |
|
|
|
|
| |
and some reformatting.
|