Conversation
martinjunger
force-pushed
the
mj-KBC-2780-basetype-to-type
branch
from
567b505 to
740ec6b
Compare
| case BaseType::INTEGER: | ||
| return new self(self::TYPE_INTEGER); | ||
| case BaseType::NUMERIC: | ||
| return new self(self::TYPE_DECIMAL, ['length' => '36,36']); |
There was a problem hiding this comment.
mozna by to nemuselo vracet celou instanci (kvuli defaulltni delce), ale jen type jako string?
Jak na tim znova premyslim, tak asi bude lepsi vracet jen ten string...
There was a problem hiding this comment.
puvodne jsem si taky myslel, ze bude fajn vracet string, ale ten UC, ktery mame na stole rika, ze chceme instanci, ne si to pak rucne tvorit v connection. Takze bych to nechal tak
Jen prosim na ty delky pouzij vyse uvedene konstanty. Zrovna ten numeric jsem nedavno upravoval
There was a problem hiding this comment.
Tyhle hodnoty byly z toho kodu UI a prislo mi to jako rozumny, mit to stejny. Ale zase kdyz mame "svoje" defaulty, tak OK.
| case BaseType::INTEGER: | ||
| return new self(self::TYPE_INTEGER); | ||
| case BaseType::NUMERIC: | ||
| return new self(self::TYPE_DECIMAL, ['length' => '36,36']); |
There was a problem hiding this comment.
puvodne jsem si taky myslel, ze bude fajn vracet string, ale ten UC, ktery mame na stole rika, ze chceme instanci, ne si to pak rucne tvorit v connection. Takze bych to nechal tak
Jen prosim na ty delky pouzij vyse uvedene konstanty. Zrovna ten numeric jsem nedavno upravoval
|
@martinjunger asi bych byl pro to prepsat do konverze (string) BASETYPE -> (string) TYPE. Tzn nevracet instanci, ale jen nazev typu. Jak rikas, stejne se to v connection pak vytahuje. A nemusime se pak zabyvat temi defaultnimi delkami |
Prepsano. |
jirkasemmler
force-pushed
the
mj-KBC-2780-basetype-to-type
branch
from
c04ba77 to
1014bb3
Compare
KBC-2780 basetype to type mapping
2 participants
Jira: KBC-2780