[1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relations

Dieser Bereich dient dazu, neue Features zu diskutieren und für die Entwicklung zu dokumentieren. // This area is dedicated to new features including proposals and documentation.
Benutzeravatar
dr.e.
Administrator
Beiträge: 4533
Registriert: 04.11.2007, 16:13:53

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von dr.e. » 26.08.2012, 14:46:49

Hallo Lutz,

danke für die Tests und die Überarbeitung! Gute Arbeit! :)
Viele Grüße,
Christian

Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von MrNiceGuy » 26.08.2012, 16:20:38

Danke :)

Ich hatte noch überlegt, ob es nicht sinnvoll wäre wenn ich den string für die sql-Anpassung als leeren string deklarieren anstatt mit null. Nunja. Das ist aber nur temporär wichtig denke ich
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4533
Registriert: 04.11.2007, 16:13:53

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von dr.e. » 26.08.2012, 22:17:36

Hmm, null lässt sich als Datentyp IMHO einfacher abfragen.
Viele Grüße,
Christian

Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von MrNiceGuy » 27.08.2012, 11:27:50

Wird es aber nicht, sondern mittels concatenation in den SQL-Query eingefügt, deshalb wäre '' statt null wohl der sinnvollere Wert!? Für den Prefix halte ich null auch für den geeigneteren Startwert.
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4533
Registriert: 04.11.2007, 16:13:53

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von dr.e. » 27.08.2012, 13:44:50

Dann gerne auch ''.
Viele Grüße,
Christian

Benutzeravatar
dr.e.
Administrator
Beiträge: 4533
Registriert: 04.11.2007, 16:13:53

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von dr.e. » 05.09.2012, 21:01:43

Hallo Lutz,

ich habe gerade in einem Projekt mit einer alten Konfiguration festgestellt, dass

Code: Alles auswählen

if (strcasecmp($this->relationTable[$relationName]['Timestamps'], 'TRUE') == 0) {
   $uniqueRelationPrefix = md5(uniqid(mt_rand(), true));
   $relationTimestamps = ', `' . $uniqueRelationSourceId . '_' . $relationTable . '`.`CreationTimestamp` AS `' . $uniqueRelationPrefix . '_CreationTimestamp`';
} 
zu einem
Error-ID:
61d51efcec95b8d036a3d2b4bf99d828
Message:
Undefined index: Timestamps
Number:
8
File:
D:\Entwicklung\SourceforgeSVN\branches\php5\1.16\modules\genericormapper\data\GenericORRelationMapper.php
Line:
517
führt. Ändere ich die Code-Stelle in

Code: Alles auswählen

if (isset($this->relationTable[$relationName]['Timestamps']) && strcasecmp($this->relationTable[$relationName]['Timestamps'], 'TRUE') == 0) {
   $uniqueRelationPrefix = md5(uniqid(mt_rand(), true));
   $relationTimestamps = ', `' . $uniqueRelationSourceId . '_' . $relationTable . '`.`CreationTimestamp` AS `' . $uniqueRelationPrefix . '_CreationTimestamp`';
} 
passt das. Kannst du den Fix bitte prüfen und bestätigen?

Danke! :)
Viele Grüße,
Christian

Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von MrNiceGuy » 07.09.2012, 16:45:20

Oh, jetzt wo du es erwähnst: Ich habe tatsächlich immer nur MIT dem Parameter (auf "TRUE" bzw. "FALSE" gesetzt) getestet... :oops: Mit der Änderung geht es dann natürlich.
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4533
Registriert: 04.11.2007, 16:13:53

Re: [1.16] Erzeugungs- und Bearbeitungsdatum für GORM-Relati

Beitrag von dr.e. » 07.09.2012, 23:22:55

Kein Ding! Dafür gibt es das 4-Augen-Prinzip. ;) Sofern die Änderung dein Feature nicht tötet ist alles gut... :D
Viele Grüße,
Christian

Gesperrt

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 1 Gast