Problém s časovým intervalem z DB

+
0
-

Zdravím!

V databázi mám časový údaj (interval) ve formátu

HH:MM:SS.ms

který ale nejde přes Nette Database Explorer načíst.
Spadne to v souboru Database\ResultSet.php na řádku 180:

<?php
178:                } elseif ($type === IStructure::FIELD_TIME_INTERVAL) {
179:                    preg_match('#^(-?)(\d+)\D(\d+)\D(\d+)\z#', $value, $m);
180:                    $row[$key] = new \DateInterval("PT$m[2]H$m[3]M$m[4]S");
181:                    $row[$key]->invert = (int) (bool) $m[1];
182:
183:                }
?>

protože předcházející preg_match tenhle formát nepozná a vrátí null. Dá se
s tím něco dělat, nějak korektně to obejít? Vřelé díky!

S úctou a pozdravem
Vladimír Pospíšil

CZechBoY
Člen | 3608
+
0
-

Poslat pull request do nette/database, který přidá volitelné milisekundy.

+
0
-

Aha, tak na to jsem coby začátečník nenašel dost drzosti… no ale zkusit to můžu.

+
0
-

No, tak potřebuji další radu. Naklonoval jsem si větev 2.4, udělal lokálně příslušné změny vč. testu a teď ale jak je pushnout na github, abych mohl pull request vytvořit? Na pushování jako běžný návštěvník nemám práva.

Ondřej Kubíček
Člen | 494
+
+1
-

musíš si napřed udělat fork, na tom udělat změny a ten pak poslat na PR

+
0
-

To mě napadlo, ale na rozhraní githubu jsem žádnou takovou možnost neobjevil a když jsem si vytvořil novou větev lokálně, stejně mi ji server nedovolil pushnout.

Ondřej Kubíček
Člen | 494
+
+1
-

je to u každého projektu vpravo nahoře https://blog.tomasfejfar.cz/…pullrequest/

+
0
-

No, nakonec jsem to našel. Mám na pull request někoho upozornit, nebo to zařídí server sám?

+
0
-

No a teď další dotaz. Neprošly testy, ačkoliv na lokále mi klaply perfektně. Je možné se nějak dostat ke kompletním výstupním souborům, abych si je mohl porovnat?

David Grudl
Nette Core | 8239
+
0
-

Ty testy neprocházejí v PHP < 7.1, protože tam ještě nativní DateInterval nemá proměnnou f. Takže buď upravit test, nebo tuhle featuru přidat jen když if (PHP_VERSION_ID >= 70100) {

+
0
-

Aha, to mi uniklo. Přidám.

CZechBoY
Člen | 3608
+
0
-

Problémy s pull requestem bych řešil spíše u daného pull requestu než na foru :-)