Merge pull request #917 from warren5236/make-track-selection-optional…

…-when-tracks-are-available Make the track selection option on the talk form.
joindin · Apr 24, 2024 · d64200b · d64200b
2 parents d0c00c0 + 797a87c
commit d64200b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/app/src/Talk/TalkController.php b/app/src/Talk/TalkController.php
@@ -200,6 +200,12 @@ public function editTalk($eventSlug, $talkSlug)
                         }
                     }
 
+                    if (empty($values['track'])) {
+                        foreach ($talk->getTracks() as $t) {
+                            $talkApi->removeTalkFromTrack($t->remove_track_uri);
+                        }
+                    }
+
                     $talkUrl = $this->application->urlFor('talk', ['eventSlug' => $eventSlug, 'talkSlug' => $talkSlug]);
                     $this->application->redirect($talkUrl);
                 } catch (\RuntimeException $e) {

diff --git a/app/src/Talk/TalkFormType.php b/app/src/Talk/TalkFormType.php
@@ -160,7 +160,7 @@ public function buildForm(FormBuilderInterface $builder, array $options)
                 'track',
                 'choice',
                 [
-                    'required' => (bool) !empty($this->tracks),
+                    'required' => false,
                     'choices'  => ['' => '']  + $this->tracks
                 ]
             )