MBS-8974: Add repeating of cards #47
Conversation
sh-csg
force-pushed
the
MBS-8974-repeating-cards
branch
2 times, most recently
from
d7efc8d to
29d3bbb
Compare
sh-csg
force-pushed
the
MBS-8974-repeating-cards
branch
from
29d3bbb to
fdcf7c8
Compare
|
Wunderbar. Codechecker & Co sind auch zufrieden.
/var/www/html/lib/phpunit/classes/util.php:317 |
|
|
||
| $mform->setType('repeat_interval', PARAM_INT); | ||
| $mform->setType('repeat_interval_type', PARAM_INT); | ||
| $mform->setDefault('repeat_interval', 1); |
There was a problem hiding this comment.
Bei mir ist der default leider 0 (also 0 Stunden).
There was a problem hiding this comment.
Das kommt wohl daher, dass der Standard-Wert in der DB 0 ist. Ich habe das angepasst.
| $dbman->add_field($table, $field); | ||
| } | ||
|
|
||
| $field = new xmldb_field( |
There was a problem hiding this comment.
Warum wird hier eine unterschiedliche Formatierung genutzt? Oben inline, hier über mehrere Zeilen.
There was a problem hiding this comment.
Wegen der maximalen Zeilenlänge von 132 Zeichen.
|
In Verbindung mit MBS-8982: Add card numbers müsste dann noch überprüft werden, ob die Nummer automatisch hochgezählt werden. |
classes/boardmanager.php
Outdated
| } | ||
| $card->repeat_enable = 0; | ||
| $this->update_card($cardid, (array) $card); | ||
| $newcard->isrepeated = 1; |
There was a problem hiding this comment.
Ist entfernt, das ist wohl ein Überrest einer alten Implementierung
|
unit test error is probably caused by another plugin in our mbsmoodle instance. |
| $card->repeat_enable = 0; | ||
| $this->update_card($cardid, (array) $card); | ||
| $newcard->isrepeated = 1; | ||
| $this->add_card($this->get_leftmost_column($card->kanban_board), 0, (array)$newcard); |
There was a problem hiding this comment.
(array) $newcard fehlendes Leerzeichen
There was a problem hiding this comment.
PHPCS moniert das nicht
Das müsste funktionieren - add_card() kümmert sich ja um das Hochzählen |
No description provided.