You are viewing a plain text version of this content. The canonical link for it is here.
Posted to user@empire-db.apache.org by Alain Becam <Al...@embl.de> on 2010/12/22 11:22:55 UTC
getTitle() or getName()
Hello,
I have just discovered EmpireDB some weeks ago, and start to use it
for our next implementation. Thank you for this nice work, I am very
impressed by such a good KISS approach (especially of course the pure
POJO, no String-literal approach, which is extremely handy for
"SQL-centric" applications).
Then I am migrating some of our table, and it appeared that a field was
null for a required columns (the primary key ;) ). And I got the
message: "The value for field null must not be null."
If I actually replace the getTitle() in DBRowSet.updateRecord by
getName() I get the column name:
else if (primaryKey!=null && primaryKey.contains(col))
{ // All primary key fields must be supplied
- return error(DBErrors.FieldNotNull,
col.getTitle());
+ return error(DBErrors.FieldNotNull,
"(updateRecord: part of primary key) " + col.getName());
}
else if (col.isRequired())
{ // Error Column is required!
- return error(DBErrors.FieldNotNull,
col.getTitle());
+ return error(DBErrors.FieldNotNull,
"(updateRecord: required) " + col.getName());
}
I wonder if there is something I am missing here.
Cheers,
Alain
Re: getTitle() or getName()
Posted by Francis De Brabandere <fr...@gmail.com>.
On Wed, Dec 22, 2010 at 3:43 PM, Alain Becam <Al...@embl.de> wrote:
> Done. Sorry I did not use Jira to report it.
No need for that, it's never a bad Idea to ask on the list first.
Thanks for updating the issue.
Francis
> On 12/22/2010 3:37 PM, Francis De Brabandere wrote:
>>
>> This is the related issue, can you add the comment?
>> https://issues.apache.org/jira/browse/EMPIREDB-93.
>>
>> If you don't have a login (or don't want to create one) let me know I'll
>> do it.
>>
>> Thanks
>
--
http://www.somatik.be
Microsoft gives you windows, Linux gives you the whole house.
Re: getTitle() or getName()
Posted by Alain Becam <Al...@embl.de>.
Done. Sorry I did not use Jira to report it.
On 12/22/2010 3:37 PM, Francis De Brabandere wrote:
> This is the related issue, can you add the comment?
> https://issues.apache.org/jira/browse/EMPIREDB-93.
>
> If you don't have a login (or don't want to create one) let me know I'll do it.
>
> Thanks
Re: getTitle() or getName()
Posted by Francis De Brabandere <fr...@gmail.com>.
This is the related issue, can you add the comment?
https://issues.apache.org/jira/browse/EMPIREDB-93.
If you don't have a login (or don't want to create one) let me know I'll do it.
Thanks
On Wed, Dec 22, 2010 at 3:34 PM, Alain Becam <Al...@embl.de> wrote:
> Thank you for this very quick feedback. You might want to change it in
> DBTableColumn.checkValue too.
>
> Alain
>
> On 12/22/2010 2:48 PM, Rainer Döbele wrote:
>>
>> Hi Alain,
>>
>> thanks for your commendation of our little project.
>> We are certainly significantly different from other database access
>> solutions and it's good to see that other people share our enthusiasm for
>> this approach.
>>
>> About the getTitle() in DBRowSet.updateRecord:
>> This is indeed a (minor) bug - thanks for reporting it.
>> The getName() function delivers the physical name whereas getTitle() is an
>> optional attribute that may be set to supply a logical name.
>>
>> I will fix this bug immediately.
>>
>> Best regards
>> Rainer
>>
>
--
http://www.somatik.be
Microsoft gives you windows, Linux gives you the whole house.
Re: getTitle() or getName()
Posted by Alain Becam <Al...@embl.de>.
Thank you for this very quick feedback. You might want to change it in
DBTableColumn.checkValue too.
Alain
On 12/22/2010 2:48 PM, Rainer Döbele wrote:
> Hi Alain,
>
> thanks for your commendation of our little project.
> We are certainly significantly different from other database access solutions and it's good to see that other people share our enthusiasm for this approach.
>
> About the getTitle() in DBRowSet.updateRecord:
> This is indeed a (minor) bug - thanks for reporting it.
> The getName() function delivers the physical name whereas getTitle() is an optional attribute that may be set to supply a logical name.
>
> I will fix this bug immediately.
>
> Best regards
> Rainer
>
re: getTitle() or getName()
Posted by Rainer Döbele <do...@esteam.de>.
Hi Alain,
thanks for your commendation of our little project.
We are certainly significantly different from other database access solutions and it's good to see that other people share our enthusiasm for this approach.
About the getTitle() in DBRowSet.updateRecord:
This is indeed a (minor) bug - thanks for reporting it.
The getName() function delivers the physical name whereas getTitle() is an optional attribute that may be set to supply a logical name.
I will fix this bug immediately.
Best regards
Rainer
Alain Becam wrote:
> from: Alain Becam [mailto:Alain.Becam@embl.de]
> to: empire-db-user@incubator.apache.org
> re: getTitle() or getName()
>
> Hello,
> I have just discovered EmpireDB some weeks ago, and start to use it
> for our next implementation. Thank you for this nice work, I am very
> impressed by such a good KISS approach (especially of course the pure
> POJO, no String-literal approach, which is extremely handy for
> "SQL-centric" applications).
>
> Then I am migrating some of our table, and it appeared that a field was
> null for a required columns (the primary key ;) ). And I got the
> message: "The value for field null must not be null."
>
> If I actually replace the getTitle() in DBRowSet.updateRecord by
> getName() I get the column name:
>
> else if (primaryKey!=null &&
> primaryKey.contains(col))
> { // All primary key fields must be supplied
> - return error(DBErrors.FieldNotNull,
> col.getTitle());
> + return error(DBErrors.FieldNotNull,
> "(updateRecord: part of primary key) " + col.getName());
> }
> else if (col.isRequired())
> { // Error Column is required!
> - return error(DBErrors.FieldNotNull,
> col.getTitle());
> + return error(DBErrors.FieldNotNull,
> "(updateRecord: required) " + col.getName());
> }
>
> I wonder if there is something I am missing here.
>
> Cheers,
> Alain