https://www.kivitendo.de/redmine/https://www.kivitendo.de/redmine/redmine/favicon.ico?15768422422016-08-20T10:59:02Zwissenprojekt kivitendo - Fehler #204: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehrhttps://www.kivitendo.de/redmine/issues/204?journal_id=2892016-08-20T10:59:02ZJan Bürenjan@kivitendo-premium.de
<ul></ul><p>Hier nochmal, die ursprüngliche Anforderung:</p>
<p><a class="external" href="http://trac.kivitendo.de/ticket/2491">http://trac.kivitendo.de/ticket/2491</a></p> projekt kivitendo - Fehler #204: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehrhttps://www.kivitendo.de/redmine/issues/204?journal_id=2972016-08-30T13:41:16ZMartin Helmlingmartin.helmling@opendynamic.de
<ul></ul><p>Habe ich gerade versucht zu reproduzieren:<br />neue Verkaufrechnung angelegt, eine Position angelegt mit Anzahl 10.<br />Dann gebucht, und unter Lagerbewegungen werden 10 Stk des Artikel herausgenommen/verschickt.</p>
<p>Was funktioniert den nicht !!</p> projekt kivitendo - Fehler #204: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehrhttps://www.kivitendo.de/redmine/issues/204?journal_id=3032016-09-05T14:50:57ZBernd Bleßmann
<ul></ul><p>Jan hat das gefixt in <a class="changeset" title="new_lastmtime um optionalen dbh erweitert" href="https://www.kivitendo.de/redmine/projects/forum/repository/kivitendo/revisions/573d7fd17b43a09a21e1acb2423b0160523819a8">573d7fd17b43a09a21e1acb2423b0160523819a8</a> und <a class="changeset" title="new_lastmtime mit $provided_dbh aufrufen fix für #204" href="https://www.kivitendo.de/redmine/projects/forum/repository/kivitendo/revisions/f5b13074ae7356ed59a529ace1ccd76c5b4e49c9">f5b13074ae7356ed59a529ace1ccd76c5b4e49c9</a>.</p>
<p>Entweder hat er vergessen, das Ticket hier zu schließen oder hat sich darauf verlassen, dass das "fix für <a class="issue tracker-1 status-3 priority-2 priority-default closed" title="Fehler: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehr (Gelöst)" href="https://www.kivitendo.de/redmine/issues/204">#204</a>" in seiner commit message das für ihn erledigt.</p> projekt kivitendo - Fehler #204: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehrhttps://www.kivitendo.de/redmine/issues/204?journal_id=3042016-09-05T16:22:41ZMartin Helmlingmartin.helmling@opendynamic.de
<ul></ul><p>Ja nun muss das Ganze wieder an single-dbh angepasst werden<br />oder ist $provided_dbh bereits das single-dbh ?</p>
<p>Werde es mal anschauen</p> projekt kivitendo - Fehler #204: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehrhttps://www.kivitendo.de/redmine/issues/204?journal_id=3052016-09-05T20:12:18ZBernd Bleßmann
<ul></ul><p>Martin Helmling schrieb:</p>
<blockquote>
<p>Ja nun muss das Ganze wieder an single-dbh angepasst werden<br />oder ist $provided_dbh bereits das single-dbh ?</p>
</blockquote>
<p>Nun, da jetzt alles single-dbh sein sollte, sollte es auch $provided_dbh sein. Und wenn nicht, ist es auf jeden Fall der Handler der aufrufenden Routine (wenn die nicht irgendwas ungewöhnlich komisches übergibt. Hier ist es aber SL::DB::Inventory->new->db->dbh).<br />Der commit hat sich wahrscheinlich durch die Umstellung auf single-dbh erledigt, da get_standard_dbh ja jetzt den Rose-Handler liefert.</p>
<p>Genau da lag ja vorher das Problem. Beim Auslagern aus der Rechnung wird post_invoice nämlich mit dem Rose-Handler (in einer Transaktion) aufgerufen, der auch das Auslagern übernimmt, um das alles in einer Transaktion zu haben. Durch Deine Änderung wurde aber für dieselbe DB-Zeile, um mtime zu lesen, ein neuer/anderer Handler (standard_dbh) verwendet. Und der wartete auf das Ende der Transaktion (und wartete, und wartete).</p> projekt kivitendo - Fehler #204: Automatisches Auslagern beim Rechnung schreiben funktioniert mit Commit 8c1d5d nicht mehrhttps://www.kivitendo.de/redmine/issues/204?journal_id=3092016-10-10T13:02:39ZJan Bürenjan@kivitendo-premium.de
<ul><li><strong>Status</strong> wurde von <i>Neu</i> zu <i>Gelöst</i> geändert</li></ul><p>Hi, ja das ist in der Tat jetzt gefixt. Ich hatte das hier nur nochmal dokumentiert.<br />Ist jetzt geschlossen</p>