[smarcet]

* fix on languages for CFP bio form

Author: smarcet

summit/code/forms/SpeakerForm.php

@@ -196,8 +196,8 @@ public function loadDataFrom($data, $mergeStrategy = 0, $fieldList = null)
             if(empty($speaker->TwitterName))  $this->fields->fieldByName('TwitterName')->setValue($speaker->Member()->TwitterName);
         }
 
-        $this->fields->fieldByName('Expertise')->setValue(implode(',',$speaker->AreasOfExpertise()->map('ID','Expertise')->toArray()));
-        $this->fields->fieldByName('Language')->setValue(implode(',',$speaker->Languages()->map('ID','Language')->toArray()));
+        $this->fields->fieldByName('Expertise')->setValue(implode(',', $speaker->AreasOfExpertise()->map('ID','Expertise')->toArray()));
+        $this->fields->fieldByName('Language')->setValue(implode(',', $speaker->Languages()->map('ID','Name')->toArray()));
 
         $country_array = array();
         foreach ($speaker->TravelPreferences() as $pref_country) {
@@ -259,8 +259,11 @@ public function saveInto(DataObjectInterface $dataObject, $fieldList = null) {
 
         $language = $this->fields->fieldByName("Language")->Value();
         $speaker->Languages()->removeAll();
-        foreach(explode(',',$language) as $lang_id) ;
-            $speaker->Languages()->add($lang_id);
+        foreach(explode(',',$language) as $lang_name) {
+            $lang = Language::get()->where(sprintf("LOWER(Name) = '%s'", strtolower($lang_name)))->first();
+            if(!$lang) continue;
+            $speaker->Languages()->add($lang);
+        }
 
         $link_ids = [];
         for($i = 1 ; $i <= 5 ; $i++ ){

summit/code/pages/PresentationPage.php

@@ -345,7 +345,6 @@ public function preview(SS_HTTPRequest $r)
         ))->renderWith(array('PresentationPage_preview', 'PresentationPage'));
     }
 
-
     /**
      * Action that shows the presentation details, readonly
      * @param   $r SS_HTTPRequest

summit/javascript/speaker-form.js

@@ -80,7 +80,7 @@ $(document).ready(function(){
                 minlength: 3,
                 name: 'languages',
                 displayKey: 'name',
-                valueKey: 'id',
+                valueKey: 'name',
                 source: languages.ttAdapter()
             }
         ]