From cf5a486588444a96e2f7ff63c84528ca35f73f9b Mon Sep 17 00:00:00 2001 From: Gustavo Martin Morcuende Date: Wed, 16 May 2012 01:31:47 +0200 Subject: [PATCH] Ads edition and creation with internationalization Nested forms in order to create or edit ads (relation 1 to N between data base tables) I hate internationalization... --- apps/companyfront/config/app.yml | 2 +- apps/companyfront/modules/ad/templates/_form.php | 41 +--------- apps/companyfront/modules/ad/templates/_list.php | 8 +- .../modules/ad/templates/editSuccess.php | 3 +- .../modules/ad/templates/indexSuccess.php | 2 +- .../modules/ad/templates/newSuccess.php | 2 +- apps/companyfront/templates/layout.php | 8 +- lib/form/doctrine/AdDescriptionForm.class.php | 8 ++ lib/form/doctrine/AdForm.class.php | 95 +++++++++++++++++++++- 9 files changed, 116 insertions(+), 53 deletions(-) diff --git a/apps/companyfront/config/app.yml b/apps/companyfront/config/app.yml index 7cad5cf..0e41bc0 100644 --- a/apps/companyfront/config/app.yml +++ b/apps/companyfront/config/app.yml @@ -4,7 +4,7 @@ # default values all: max_offices_on_pager: 3 - max_ads_on_pager: 3 + max_ads_on_pager: 2 sf_guard_plugin: remember_key_expiration_age: 2592000 # 30 days in seconds diff --git a/apps/companyfront/modules/ad/templates/_form.php b/apps/companyfront/modules/ad/templates/_form.php index 055a634..66bba95 100644 --- a/apps/companyfront/modules/ad/templates/_form.php +++ b/apps/companyfront/modules/ad/templates/_form.php @@ -11,50 +11,13 @@ renderHiddenFields(false) ?>  Back to list - getObject()->isNew()): ?> -  getObject()->getId(), array('method' => 'delete', 'confirm' => 'Are you sure?')) ?> - - + renderGlobalErrors() ?> - - renderLabel() ?> - - renderError() ?> - - - - - renderLabel() ?> - - renderError() ?> - - - - - renderLabel() ?> - - renderError() ?> - - - - - renderLabel() ?> - - renderError() ?> - - - - - renderLabel() ?> - - renderError() ?> - - - + diff --git a/apps/companyfront/modules/ad/templates/_list.php b/apps/companyfront/modules/ad/templates/_list.php index 1ddfa3d..09e66bd 100644 --- a/apps/companyfront/modules/ad/templates/_list.php +++ b/apps/companyfront/modules/ad/templates/_list.php @@ -1,7 +1,7 @@ - + @@ -17,11 +17,11 @@ - + - - + + diff --git a/apps/companyfront/modules/ad/templates/editSuccess.php b/apps/companyfront/modules/ad/templates/editSuccess.php index b6d77be..0cdbaf0 100644 --- a/apps/companyfront/modules/ad/templates/editSuccess.php +++ b/apps/companyfront/modules/ad/templates/editSuccess.php @@ -1,3 +1,4 @@ -

Edit Ad

+

+ $form)) ?> diff --git a/apps/companyfront/modules/ad/templates/indexSuccess.php b/apps/companyfront/modules/ad/templates/indexSuccess.php index 48948e2..5c2dfd1 100644 --- a/apps/companyfront/modules/ad/templates/indexSuccess.php +++ b/apps/companyfront/modules/ad/templates/indexSuccess.php @@ -34,5 +34,5 @@ - + diff --git a/apps/companyfront/modules/ad/templates/newSuccess.php b/apps/companyfront/modules/ad/templates/newSuccess.php index 3d12b7f..2c9e131 100644 --- a/apps/companyfront/modules/ad/templates/newSuccess.php +++ b/apps/companyfront/modules/ad/templates/newSuccess.php @@ -1,3 +1,3 @@ -

New Ad

+

$form)) ?> diff --git a/apps/companyfront/templates/layout.php b/apps/companyfront/templates/layout.php index 9302c0e..19ee11e 100644 --- a/apps/companyfront/templates/layout.php +++ b/apps/companyfront/templates/layout.php @@ -61,7 +61,7 @@
  • Manage Ads @@ -78,13 +78,13 @@
    - Ads + Ads
    - Offices + Offices
    • Offices Index
      • diff --git a/lib/form/doctrine/AdDescriptionForm.class.php b/lib/form/doctrine/AdDescriptionForm.class.php index b6e0c1c..f2c17d0 100644 --- a/lib/form/doctrine/AdDescriptionForm.class.php +++ b/lib/form/doctrine/AdDescriptionForm.class.php @@ -12,5 +12,13 @@ class AdDescriptionForm extends BaseAdDescriptionForm { public function configure() { + //$this->useFields(array('language_id', 'ad_id', 'ad_name', 'ad_description', 'ad_mobile_text', 'ad_link')); + unset($this['ad_id']); + + if ($this->object->exists()) + { + $this->widgetSchema['delete'] = new sfWidgetFormInputCheckbox(); + $this->validatorSchema['delete'] = new sfValidatorPass(); + } } } diff --git a/lib/form/doctrine/AdForm.class.php b/lib/form/doctrine/AdForm.class.php index c5e32ef..53d3c48 100644 --- a/lib/form/doctrine/AdForm.class.php +++ b/lib/form/doctrine/AdForm.class.php @@ -5,12 +5,103 @@ * * @package mobiads * @subpackage form - * @author Your name here - * @version SVN: $Id: sfDoctrineFormTemplate.php 23810 2009-11-12 11:07:44Z Kris.Wallsmith $ + * @author Gustavo Martin Morcuende + * @version */ class AdForm extends BaseAdForm { + protected $scheduledForDeletion = array(); + public function configure() { + $this->useFields(array('company_categ_id', 'ad_mobile_image_link')); + + // Ad creation form + $adDescription = new AdDescription(); + $adDescription->Ad = $this->getObject(); + $newAdDescriptionForm = new AdDescriptionForm($adDescription); + + $this->embedForm('new', $newAdDescriptionForm); + + $this->embedRelation('AdDescription'); + + } + + protected function doBind(array $values) + { + if ('' === trim($values['new']['ad_description']) && '' === trim($values['new']['ad_mobile_text']) + && '' === trim($values['new']['ad_name']) && '' === trim($values['new']['ad_link'])) + { + unset($values['new'], $this['new']); + } + + if (isset($values['AdDescription'])) + { + foreach ($values['AdDescription'] as $i => $adDescriptionValues) + { + if (isset($adDescriptionValues['delete']) && $adDescriptionValues['id']) + { + $this->scheduledForDeletion[$i] = $adDescriptionValues['id']; + } + } + } + + + parent::doBind($values); + } + + /** + * Updates object with provided values, dealing with evantual relation deletion + * + * @see sfFormDoctrine::doUpdateObject() + */ + protected function doUpdateObject($values) + { + if (count($this->scheduledForDeletion)) + { + foreach ($this->scheduledForDeletion as $index => $id) + { + unset($values['AdDescription'][$index]); + unset($this->object['AdDescription'][$index]); + Doctrine::getTable('AdDescription')->findOneById($id)->delete(); + } + } + + $this->getObject()->fromArray($values); + } + + /** + * Saves embedded form objects. + * + * @param mixed $con An optional connection object + * @param array $forms An array of forms + */ + public function saveEmbeddedForms($con = null, $forms = null) + { + if (null === $con) + { + $con = $this->getConnection(); + } + + if (null === $forms) + { + $forms = $this->embeddedForms; + } + + foreach ($forms as $form) + { + if ($form instanceof sfFormObject) + { + if (!in_array($form->getObject()->getId(), $this->scheduledForDeletion)) + { + $form->saveEmbeddedForms($con); + $form->getObject()->save($con); + } + } + else + { + $this->saveEmbeddedForms($con, $form->getEmbeddedForms()); + } + } } } -- 2.1.4
    • getAd()->getAdMobileImageLink() ?><?php echo $ad->getAdName() ?> getAd()->getCompanyCategId() ?> getAdName() ?>', 'ad/delete?id='.$ad->getId(), array('method' => 'delete', 'confirm' => 'Are you sure?')) ?>', 'ad/delete?id='.$ad->getAd()->getId(), array('method' => 'delete', 'confirm' => 'Are you sure?')) ?>