onPHP / onphp-framework

onPHP is the mature GPL'ed multi-purpose object-oriented PHP framework.
85 stars 52 forks source link

Del #239

Closed glushchenko closed 7 years ago

DeryabinSergey commented 7 years ago

Я с подобным багом сталкивался и забил костыль вот сюда core/DB/Sequenceless.class.php вот такого вида:

                final public function query(Query $query)
                {
-                       $result = $this->queryRaw(
+                       /*$result = $this->queryRaw(
                                $query->toDialectString($this->getDialect())
-                       );
+                       );*/

+                        try {
+                            $result = $this->queryRaw(
+                                $query->toDialectString($this->getDialect())
+                            );
+                        } catch(DatabaseException $e) {
+                            if (
+                                ($query instanceof InsertQuery)
+                                && !empty($this->sequencePool[$name = $query->getTable().'_id'])
+                            ) {
+                                unset(
+                                    $this->sequencePool[
+                                        $name
+                                    ][
+                                        key($this->sequencePool[$name])
+                                    ]
+                                );
+                            }
+
+                            throw $e;
+                        }
glushchenko commented 7 years ago

Та не, я про другое, но фикс лить не буду, так как на пулл риквесты всем здесь насрать. В форке исправление тут https://github.com/bis-gmbh/onphp-framework/pull/3

suquant commented 7 years ago

@glushchenko привет, думаю стоило бы какие-то интеграции с https://travis-ci.org/ наладить. В этом MR дофигища изменений и не понятно на что они могут повлиять, даже при ревью галазами. Хотя-бы чтобы было видно что тесты проходят.

Я сам уже давно на PHP не пишу и думаю надо мнение остальных коллег выслушать по этому поводу.

glushchenko commented 7 years ago

Ребят, я просто промахнулся с пулл риквестом, хотел в наш форк, получилось сюда все наши наработки, забейте.

Проблема старая, фиксится, но оно никому не нужно.