winix_fullmorm #4

2021-05-27T10:49:34+02:00

tomasz.sowa commented

2021-05-27 10:49:34 +02:00

merge winix_fullmorm branch:

changed the way home table names are set
field() methods now takes FT field parameters (instead of booleans)
field method don't make a test for 'long format' now
fixes in escaping table names in Finder
added basic support for making migrations
has_primary_key flag in Model (I am not sure if this will be changed)

merge winix_fullmorm branch: - changed the way home table names are set - field() methods now takes FT field parameters (instead of booleans) - field method don't make a test for 'long format' now - fixes in escaping table names in Finder - added basic support for making migrations - has_primary_key flag in Model (I am not sure if this will be changed)

tomasz.sowa added 27 commits 2021-05-27 10:49:35 +02:00

Added flag has_primary_key_set to Model 133a45c84b

Now we know whether the primary key is defined or not
and we do not allow to make update/remove if the key is not defined.

And when doing insert/update we can put NULL if child models don't have
the primary key set (fields with has_foreign_key set to true).

Now in after_select() we should also set has_primary_key_set flag
or just call get_last_sequence_for_primary_key instead of get_last_sequence.

fixed: added prefix +00 when serializing PT::Date to PostgreSQL (time zone)
(for a column with a time zone there was a wrong value saved)

added FT class which is used in Model::field() methods fcf1d28b18

FT class has following types:
   enum FieldType
   {
	default_type = 0,
        primary_key = 1,
	foreign_key = 2,
	foreign_key_in_child = 4,
	no_insertable = 8,
	no_updatable = 16,
	no_fetchable = 32, /* not supported yet */
   };
an object of FT class are now used in Model::field() methods instead of insertable/updatable/is_primary_key/... boolean flags

changed the semantic of has_foreign_key (which was a bool) flag in child Models:
now on Models and list/vector of Models you should use either FT::foreign_key or FT::foreign_key_in_child
1. FT::foreign_key means that field with this flag is a foreign key and is pointing to the child object
   (it was the case when has_foreign_key was equal to true beforehand)
2. FT::foreign_key_in child means that the foreign key is in the child object and is pointing to the parent object

changed the way how the table name is set in a Model - added prepare_table() method f7490594ad

removed from Model:
virtual void table_name(PT::TextStream & stream);

added to Model:
virtual void prepare_table();
virtual void table(const wchar_t * table_name);
virtual void table(const wchar_t * schema_name, const wchar_t * table_name);

added support for PT::Space as a field in a Model 7bef1d5ead

methods before_field_value_string() and after_field_value_string() moved
from DbExpression and JsonExpression to BaseExpression and made virtual
and now methods before_field_value(const std::wstring &) and after_field_value(const std::wstring &)
(and the rest of them with string arguments) can be removed from DbExpression, PostgreSqlExpression and JsonExpression

make depend 1e2cbad6a7

fixed: prepare_to_select() should initialize 'model' but it was initialized only in select() d78aa325d3

updated to the new api of Space from pikotools c85165b442

make depend 79fd642ef7

fixed: add_field_for_select from Model incorrectly escaped a field string (column name) 9a4fd9b050

fixed: Finder didn't use full table name (schema.table) in "from" clause

fixed: a null was printed for id when serializing lists/vectors 1c5d32551a

in BaseExpression::put_field_value_list(): has_primary_key_set flag was not copied to model_env

use prepare_table() only with a database and not for flat strings c87afb40d2

Model::prepare_table() will create a log: "you should provide the table name...."
and it is not needed if a Model object is used only for flat strings

escape table names in Finder (select sql statement) 2afe111c57

WIP: #2

added support for hex strings and binary strings 009955a0fd

added FT::hexadecimal, FT::binary and FT::dont_use_utf8

fixed #2 : escape tables/columns names in Finder left join queries c7797ff2f1

some methods moved from model.h to model.cpp and from baseexpression.h to baseexpression.cpp

FT field_type changed to const FT & field_type in functions arguments aadc5be350

change FT field_type in PostgreSQLConnector 179be2864f

BaseExpression: removed methods: put_long_field_name() and put_short_field_name() a1537cf8d5

now method put_field_name() is not making a test whether field_name is in long format
and only puts the field name (without table name),
this allows us to have a dot in the column name (field_name)

changed the way how to quote schemas, tables and field names, 2ad6c8c258

added escaping table names in insert/update/remove

removed methods from BaseExpression:
virtual void before_short_field_name();
virtual void after_short_field_name();
virtual void before_first_part_long_field_name();
virtual void after_first_part_long_field_name();
virtual void before_second_part_long_field_name();
virtual void after_second_part_long_field_name();

added methods to BaseExpression:
virtual void schema_table_separator();
virtual void table_field_separator();
virtual void before_schema_name();
virtual void after_schema_name();
virtual void before_table_name();
virtual void after_table_name();
virtual void before_field_name();
virtual void after_field_name();

renamed in Model: map_fields() to fields(), prepare_table() to table() 6eaa9088e5

added to Finder: methods eq() and similar with table_name as an argument de4abeb91c

Finder<ModelClass> & eq(const wchar_t * table_name, const wchar_t * field_name, const FieldValue & field_value)
Finder<ModelClass> & eq(const wchar_t * table_name, int table_index, const wchar_t * field_name, const FieldValue & field_value)
and similar for neq(), lt(), gt(), le(), ge() and in()

added basic support for making migrations b12037a7e5

namespace PT renamed to pt 34274ca230

updated to the new pikotools api: changed utf8 functions PascalCase to snake_case 0ff900f626

updated to the new pikotools api: SpaceParser::SetSpace(...) methods have been removed 9598cc4def

updated to the new pikotools api: snake_case names of methods from SpaceParser fcd2c4775b

fixed: such field types: no_insertable, no_updatable were not taken into account on Model child objects e0e3465673

added: field type: no_removable - it is used only with child Models objects

fixed: has_primary_key_set was always set to false in Model::insert() 2f72bd29ec

and this prevented to insert a model which had a primary key set by hand

tomasz.sowa merged commit 284cbc5c66 into master

2021-05-27 10:49:46 +02:00

tomasz.sowa referenced this issue from a commit

2021-05-27 10:49:48 +02:00

Merge pull request 'winix_fullmorm' (#4) from winix_fullmorm into master

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: tomasz.sowa/morm#4