Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 868de1e

Browse filesBrowse files
weaverryanweaverryan
committed
minor #3698 Dynamic form modification cookbook: Fix inclusion of code (michaelperrin)
This PR was merged into the 2.3 branch. Discussion ---------- Dynamic form modification cookbook: Fix inclusion of code Unfortunately, reStructuredText doesn't support inclusion of code inside a code block, resulting to non-interpreted code (the `include` keyword is not displayed) as it happens on http://symfony.com/doc/2.3/cookbook/form/dynamic_form_modification.html . I tried in many ways to avoid this, but didn't find any nice way. I had to duplicate code instead. The PR includes a few minor fixes on code style as well. Commits ------- 1e8b0d4 Small fixes in code style 8093719 Dynamic form modification cookbook: Fix inclusion of code
2 parents acf255d + 1e8b0d4 commit 868de1e
Copy full SHA for 868de1e

File tree

Expand file treeCollapse file tree

2 files changed

+54
-30
lines changed
Filter options
Expand file treeCollapse file tree

2 files changed

+54
-30
lines changed

‎cookbook/form/dynamic_form_modification.rst

Copy file name to clipboardExpand all lines: cookbook/form/dynamic_form_modification.rst
+54-5Lines changed: 54 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ the event listener might look like the following::
116116
public function buildForm(FormBuilderInterface $builder, array $options)
117117
{
118118
// ...
119-
$builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event){
119+
$builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
120120
$product = $event->getData();
121121
$form = $event->getForm();
122122

@@ -147,7 +147,8 @@ the event listener might look like the following::
147147
$builder->addEventListener(FormEvents::PRE_SET_DATA, array($this, 'onPreSetData'));
148148
}
149149

150-
public function onPreSetData(FormEvent $event){
150+
public function onPreSetData(FormEvent $event)
151+
{
151152
// ...
152153
}
153154
}
@@ -253,7 +254,7 @@ Using an event listener, your form might look like this::
253254
->add('subject', 'text')
254255
->add('body', 'textarea')
255256
;
256-
$builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event){
257+
$builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
257258
// ... add a choice list of friends of the current application user
258259
});
259260
}
@@ -653,7 +654,31 @@ field according to the current selection in the ``sport`` field:
653654
{# ... #}
654655
{{ form_end(form) }}
655656

656-
.. include:: /cookbook/form/dynamic_form_modification_ajax_js.rst.inc
657+
<script>
658+
var $sport = $('#meetup_sport');
659+
// When sport gets selected ...
660+
$sport.change(function() {
661+
// ... retrieve the corresponding form.
662+
var $form = $(this).closest('form');
663+
// Simulate form data, but only include the selected sport value.
664+
var data = {};
665+
data[$sport.attr('name')] = $sport.val();
666+
// Submit data via AJAX to the form's action path.
667+
$.ajax({
668+
url : $form.attr('action'),
669+
type: $form.attr('method'),
670+
data : data,
671+
success: function(html) {
672+
// Replace current position field ...
673+
$('#meetup_position').replaceWith(
674+
// ... with the returned one from the AJAX response.
675+
$(html).find('#meetup_position')
676+
);
677+
// Position field now displays the appropriate positions.
678+
}
679+
});
680+
});
681+
</script>
657682

658683
.. code-block:: html+php
659684

@@ -664,7 +689,31 @@ field according to the current selection in the ``sport`` field:
664689
<!-- ... -->
665690
<?php echo $view['form']->end($form) ?>
666691

667-
.. include:: /cookbook/form/dynamic_form_modification_ajax_js.rst.inc
692+
<script>
693+
var $sport = $('#meetup_sport');
694+
// When sport gets selected ...
695+
$sport.change(function() {
696+
// ... retrieve the corresponding form.
697+
var $form = $(this).closest('form');
698+
// Simulate form data, but only include the selected sport value.
699+
var data = {};
700+
data[$sport.attr('name')] = $sport.val();
701+
// Submit data via AJAX to the form's action path.
702+
$.ajax({
703+
url : $form.attr('action'),
704+
type: $form.attr('method'),
705+
data : data,
706+
success: function(html) {
707+
// Replace current position field ...
708+
$('#meetup_position').replaceWith(
709+
// ... with the returned one from the AJAX response.
710+
$(html).find('#meetup_position')
711+
);
712+
// Position field now displays the appropriate positions.
713+
}
714+
});
715+
});
716+
</script>
668717

669718
The major benefit of submitting the whole form to just extract the updated
670719
``position`` field is that no additional server-side code is needed; all the

‎cookbook/form/dynamic_form_modification_ajax_js.rst.inc

Copy file name to clipboardExpand all lines: cookbook/form/dynamic_form_modification_ajax_js.rst.inc
-25Lines changed: 0 additions & 25 deletions
This file was deleted.

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.