Update career extra fields BT#16231

main/admin/careers.php

@@ -155,6 +155,11 @@ switch ($action) {
                 $career->update_all_promotion_status_by_career_id($values['id'], $values['status']);
                 $old_status = $career->get_status($values['id']);
                 $res = $career->update($values);
+
+                $values['item_id'] = $values['id'];
+                $sessionFieldValue = new ExtraFieldValue('career');
+                $sessionFieldValue->saveFieldValues($values);
+
                 if ($res) {
                     Display::addFlash(
                         Display::return_message(get_lang('CareerUpdated'), 'confirmation')

main/inc/lib/career.lib.php

@@ -128,7 +128,7 @@ class Career extends Model
             $header = get_lang('Modify');
         }
 
-        $id = isset($_GET['id']) ? intval($_GET['id']) : '';
+        $id = isset($_GET['id']) ? (int) $_GET['id'] : '';
         $form->addHeader($header);
         $form->addHidden('id', $id);
         $form->addElement('text', 'name', get_lang('Name'), ['size' => '70']);
@@ -145,14 +145,16 @@ class Career extends Model
         );
         $status_list = $this->get_status_list();
         $form->addElement('select', 'status', get_lang('Status'), $status_list);
+
         if ($action == 'edit') {
+            $extraField = new ExtraField('career');
+            $extraField->addElements($form, $id);
+
             $form->addElement('text', 'created_at', get_lang('CreatedAt'));
             $form->freeze('created_at');
-        }
+            $form->addButtonSave(get_lang('Modify'));
-        if ($action == 'edit') {
-            $form->addButtonSave(get_lang('Modify'), 'submit');
         } else {
-            $form->addButtonCreate(get_lang('Add'), 'submit');
+            $form->addButtonCreate(get_lang('Add'));
         }
 
         // Setting the defaults
@@ -167,6 +169,8 @@ class Career extends Model
 
         $form->setDefaults($defaults);
 
+
+
         // Setting the rules
         $form->addRule('name', get_lang('ThisFieldIsRequired'), 'required');