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 db40268..dc84d4e 100644
--- a/app/views/module/module/modul.php
+++ b/app/views/module/module/modul.php
@@ -458,7 +458,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 a3a0a92..dfbb573 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 7416e18..efc2590 100644
--- a/app/views/search/studiengaenge/verlauf.php
+++ b/app/views/search/studiengaenge/verlauf.php
@@ -68,7 +68,7 @@
                             <tr class="header-row">
                                 <? $displayed_abschnitt_name = true ?>
                                 <th class="toggle-indicator" colspan="2">
-                                    <a class="toggler" href="#"><?= htmlReady($abschnitt['name']) ?><?= $abschnitt['credit_points'] ? $abschnitt['credit_points'] . ' ' . _('CP') : '' ?></a>
+                                    <a class="toggler" href="#"><?= htmlReady($abschnitt['name']) ?><?= $abschnitt['credit_points'] ? ' ' . $abschnitt['credit_points'] . ' ' . _('CP') : '' ?></a>
                                 </th>
                                 <th>
                                     <a data-dialog title="<?= sprintf(_('%s (Kommentar)'), htmlReady($abschnitt['name'])) ?>" href="<?= $controller->link_for('search/studiengaenge/kommentar', $abschnitt_id) ?>">
diff --git a/app/views/studiengaenge/versionen/abschnitt.php b/app/views/studiengaenge/versionen/abschnitt.php
index 67a4f00..0693246 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' => '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);
+    }
+}