Resolve "MVV-Studiengangsteil-Abschnitt Bearbeiten"

Closes #6172

Merge request studip/studip!4690

5 files changed, 45 insertions, 13 deletions

diff --git a/app/views/module/module/modul.php b/app/views/module/module/modul.php
index 0797d32..747b183 100644
--- a/app/views/module/module/modul.php
+++ b/app/views/module/module/modul.php
@@ -438,7 +438,7 @@ if ($GLOBALS['MVV_MODUL']['SPRACHE']['default'] != $display_language) {
 
         <label id="mvv-field-modul-kp"><?= _('Kreditpunkte') ?>
         <? if ($def_lang) : ?>
-            <input <?= $perm->disable('kp') ?> type="text" name="kp" id="kp" value="<?= htmlReady($modul->kp) ?>" maxlength="10">
+            <input <?= $perm->disable('kp') ?> type="number" step="0.01" name="kp" id="kp" value="<?= htmlReady($modul->kp) ?>" max="99" min="0">
         <? else : ?>
             <?= $modul->kp ? htmlReady($modul->kp) : _('keine Angabe') ?>
         <? endif; ?>
diff --git a/app/views/module/module/modulteil.php b/app/views/module/module/modulteil.php
index a27d233..16cc8cf 100644
--- a/app/views/module/module/modulteil.php
+++ b/app/views/module/module/modulteil.php
@@ -211,16 +211,13 @@ if ($GLOBALS['MVV_MODULTEIL']['SPRACHE']['default'] != $display_language) {
 
         <label>
             <?= _('Kreditpunkte') ?>
+            <? if ($def_lang) : ?>
+                <input <?= $perm->disable('kp') ?>
+                        type="number" name="kp" step="0.01" id="kp" value="<?= htmlReady($modulteil->kp) ?>" max="99" min="0">
+            <? else : ?>
+                <?= $modulteil->kp ? htmlReady($modulteil->kp) : _('keine Angabe') ?>
+            <? endif; ?>
         </label>
-        <? if ($def_lang) : ?>
-            <input <?= $perm->disable('kp') ?>
-                    type="text" name="kp" id="kp" value="<?= htmlReady($modulteil->kp) ?>" maxlength="10">
-        <? else : ?>
-            <?= $modulteil->kp ? htmlReady($modulteil->kp) : _('keine Angabe') ?>
-        <? endif; ?>
-
-
-
     </fieldset>
 
 
diff --git a/app/views/search/studiengaenge/verlauf.php b/app/views/search/studiengaenge/verlauf.php
index f2db804..892cfab 100644
--- a/app/views/search/studiengaenge/verlauf.php
+++ b/app/views/search/studiengaenge/verlauf.php
@@ -58,7 +58,7 @@
                             <? if (!$displayedAbschnittName) : ?>
                                 <? $displayedAbschnittName = true ?>
                                 <td rowspan="<?= $abschnitt['rowspan'] ?: 1 ?>">
-                                    <?= htmlReady($abschnitt['name']) ?><br/><?= $abschnitt['creditPoints'] ? $abschnitt['creditPoints'] . ' ' . _('CP') : '' ?>
+                                    <?= htmlReady($abschnitt['name']) ?><br/><?= $abschnitt['creditPoints'] ? ' ' . $abschnitt['creditPoints'] . ' ' . _('CP') : '' ?>
                                     <? if (trim($abschnitt['kommentar'])) : ?>
                                         <a data-dialog title="<?= sprintf(_('%s (Kommentar)'), htmlReady($abschnitt['name'])) ?>" href="<?= $controller->link_for('search/studiengaenge/kommentar', $abschnitt_id) ?>">
                                             <?= Icon::create('item', Icon::ROLE_CLICKABLE, ['title' => _('Zusatzinformationen zum Studiengangabschnitt')]) ?>
diff --git a/app/views/studiengaenge/versionen/abschnitt.php b/app/views/studiengaenge/versionen/abschnitt.php
index 41132f6..74437c1 100644
--- a/app/views/studiengaenge/versionen/abschnitt.php
+++ b/app/views/studiengaenge/versionen/abschnitt.php
@@ -21,8 +21,8 @@ $perm = MvvPerm::get($abschnitt)
             <?= MvvI18N::textarea('kommentar', $abschnitt->kommentar, ['class' => 'add_toolbar resizable ui-resizable'])->checkPermission($abschnitt) ?>
             <label><?= _('Kreditpunkte') ?>
                 <input <?= $perm->disable('kp') ?>
-                        type="text" name="kp" id="kp"
-                        value="<?= htmlReady($abschnitt->kp) ?>" size="5" maxlength="10">
+                        type="number" step="0.01" name="kp" id="kp"
+                        value="<?= htmlReady($abschnitt->kp) ?>" max="999" min="0">
             </label>
             <label><?= _('Zwischenüberschrift') ?>
                 <?= MvvI18N::input('ueberschrift', $abschnitt->ueberschrift, ['maxlength' => '254'])->checkPermission($abschnitt) ?>
diff --git a/db/migrations/5.4.22_biest6172_remove_float_precision_from_cp.php b/db/migrations/5.4.22_biest6172_remove_float_precision_from_cp.php
new file mode 100644
index 0000000..59730ff
--- /dev/null
+++ b/db/migrations/5.4.22_biest6172_remove_float_precision_from_cp.php
@@ -0,0 +1,35 @@
+<?php
+
+class Biest6172RemoveFloatPrecisionFromCp extends Migration
+{
+    public function description()
+    {
+        return 'Removes the precision from float.';
+    }
+
+    public function up()
+    {
+        $query = 'ALTER TABLE `mvv_stgteilabschnitt`
+                  CHANGE `kp` `kp` FLOAT NULL DEFAULT NULL';
+        DBManager::get()->execute($query);
+        $query = 'ALTER TABLE `mvv_modul`
+                  CHANGE `kp` `kp` FLOAT NULL DEFAULT NULL';
+        DBManager::get()->execute($query);
+        $query = 'ALTER TABLE `mvv_modulteil`
+                  CHANGE `kp` `kp` FLOAT NULL DEFAULT NULL';
+        DBManager::get()->execute($query);
+    }
+
+    public function down()
+    {
+        $query = 'ALTER TABLE `mvv_stgteilabschnitt`
+                  CHANGE `kp` `kp` DOUBLE(5,2) NULL DEFAULT NULL';
+        DBManager::get()->execute($query);
+        $query = 'ALTER TABLE `mvv_modul`
+                  CHANGE `kp` `kp` DOUBLE(5,2) NULL DEFAULT NULL';
+        DBManager::get()->execute($query);
+        $query = 'ALTER TABLE `mvv_modulteil`
+                  CHANGE `kp` `kp` DOUBLE(5,2) NULL DEFAULT NULL';
+        DBManager::get()->execute($query);
+    }
+}