newsletter modification from account

email modification from account
2013-10-24 12:10:35 +02:00
parent b9e5b4d9c7
commit ca03e374f2
7 changed files with 82 additions and 32 deletions
--- a/core/lib/Thelia/Action/Customer.php
+++ b/core/lib/Thelia/Action/Customer.php
@@ -71,7 +71,7 @@ class Customer extends BaseAction implements EventSubscriberInterface
            ->setTitleId($event->getTitle())
            ->setFirstname($event->getFirstname())
            ->setLastname($event->getLastname())
-            ->setEmail($event->getEmail())
+            ->setEmail($event->getEmail(), true)
            ->setPassword($event->getPassword())
            ->setReseller($event->getReseller())
            ->setSponsor($event->getSponsor())
--- a/core/lib/Thelia/Action/Newsletter.php
+++ b/core/lib/Thelia/Action/Newsletter.php
@@ -26,6 +26,7 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Thelia\Core\Event\Newsletter\NewsletterEvent;
 use Thelia\Core\Event\TheliaEvents;
 use Thelia\Action\BaseAction;
+use Thelia\Model\NewsletterQuery;
 use Thelia\Model\Newsletter as NewsletterModel;


@@ -51,7 +52,20 @@ class Newsletter extends BaseAction implements EventSubscriberInterface

    public function unsubscribe(NewsletterEvent $event)
    {
-        // TODO
+        if(null !== $nl = NewsletterQuery::create()->findPk($event->getId())) {
+            $nl->delete();
+        }
+    }
+
+    public function update(NewsletterEvent $event)
+    {
+        if(null !== $nl = NewsletterQuery::create()->findPk($event->getId())) {
+            $nl->setEmail($event->getEmail())
+                ->setFirstname($event->getFirstname())
+                ->setLastname($event->getLastname())
+                ->setLocale($event->getLocale())
+                ->save();
+        }
    }

    /**
@@ -78,6 +92,7 @@ class Newsletter extends BaseAction implements EventSubscriberInterface
    {
        return array(
            TheliaEvents::NEWSLETTER_SUBSCRIBE => array('subscribe', 128),
+            TheliaEvents::NEWSLETTER_UPDATE => array('update', 128),
            TheliaEvents::NEWSLETTER_UNSUBSCRIBE => array('unsubscribe', 128)
        );
    }
--- a/core/lib/Thelia/Controller/Front/CustomerController.php
+++ b/core/lib/Thelia/Controller/Front/CustomerController.php
@@ -25,6 +25,7 @@ namespace Thelia\Controller\Front;
 use Thelia\Core\Event\Customer\CustomerCreateOrUpdateEvent;
 use Thelia\Core\Event\Customer\CustomerLoginEvent;
 use Thelia\Core\Event\LostPasswordEvent;
+use Thelia\Core\Event\Newsletter\NewsletterEvent;
 use Thelia\Core\Security\Authentication\CustomerUsernamePasswordFormAuthenticator;
 use Thelia\Core\Security\Exception\AuthenticationException;
 use Thelia\Core\Security\Exception\UsernameNotFoundException;
@@ -146,7 +147,7 @@ class CustomerController extends BaseFrontController
            'firstname'    => $customer->getFirstName(),
            'lastname'     => $customer->getLastName(),
            'email'        => $customer->getEmail(),
-            'newsletter'   => NewsletterQuery::isSubscribed($customer->getEmail())
+            'newsletter'   => null !== NewsletterQuery::create()->findOneByEmail($customer->getEmail()),
        );

        $customerProfilUpdateForm = new CustomerProfilUpdateForm($this->getRequest(), 'form', $data);
@@ -203,6 +204,7 @@ class CustomerController extends BaseFrontController

            try {
                $customer = $this->getSecurityContext()->getCustomerUser();
+                $newsletterOldEmail = $customer->getEmail();

                $form = $this->validateForm($customerProfilUpdateForm, "post");

@@ -210,14 +212,29 @@ class CustomerController extends BaseFrontController
                $customerChangeEvent->setCustomer($customer);
                $this->dispatch(TheliaEvents::CUSTOMER_UPDATEPROFIL, $customerChangeEvent);

+                $updatedCustomer = $customerChangeEvent->getCustomer();
+
                // Newsletter
-                if ($form->get('newsletter')->getData()){
-                    //$this->dispatch(TheliaEvents::NEWSLETTER_SUBSCRIBE, $customerChangeEvent);
+                if (true === $form->get('newsletter')->getData()) {
+                    $nlEvent = new NewsletterEvent($updatedCustomer->getEmail(), $this->getRequest()->getSession()->getLang()->getLocale());
+                    $nlEvent->setFirstname($updatedCustomer->getFirstname());
+                    $nlEvent->setLastname($updatedCustomer->getLastname());
+
+                    if(null !== $newsletter = NewsletterQuery::create()->findOneByEmail($newsletterOldEmail)) {
+                        $nlEvent->setId($newsletter->getId());
+                        $this->dispatch(TheliaEvents::NEWSLETTER_UPDATE, $nlEvent);
                    } else {
-                    //$this->dispatch(TheliaEvents::NEWSLETTER_UNSUBSCRIBE, $customerChangeEvent);
+                        $this->dispatch(TheliaEvents::NEWSLETTER_SUBSCRIBE, $nlEvent);
+                    }
+                } else {
+                    if(null !== $newsletter = NewsletterQuery::create()->findOneByEmail($newsletterOldEmail)) {
+                        $nlEvent = new NewsletterEvent($updatedCustomer->getEmail(), $this->getRequest()->getSession()->getLang()->getLocale());
+                        $nlEvent->setId($newsletter->getId());
+                        $this->dispatch(TheliaEvents::NEWSLETTER_UNSUBSCRIBE, $nlEvent);
+                    }
                }

-                $this->processLogin($customerChangeEvent->getCustomer());
+                $this->processLogin($updatedCustomer);

                $this->redirectSuccess($customerProfilUpdateForm);

--- a/core/lib/Thelia/Core/Event/Newsletter/NewsletterEvent.php
+++ b/core/lib/Thelia/Core/Event/Newsletter/NewsletterEvent.php
@@ -32,6 +32,11 @@ use Thelia\Core\Event\ActionEvent;
 */
 class NewsletterEvent extends ActionEvent
 {
+    /**
+     * @var string email to save
+     */
+    protected $id;
+
    /**
     * @var string email to save
     */
@@ -138,6 +143,22 @@ class NewsletterEvent extends ActionEvent
        return $this->locale;
    }

+    /**
+     * @param string $id
+     */
+    public function setId($id)
+    {
+        $this->id = $id;
+    }
+
+    /**
+     * @return string
+     */
+    public function getId()
+    {
+        return $this->id;
+    }
+



--- a/core/lib/Thelia/Core/Event/TheliaEvents.php
+++ b/core/lib/Thelia/Core/Event/TheliaEvents.php
@@ -691,9 +691,6 @@ final class TheliaEvents
     * sent for subscribing to the newsletter
     */
    const NEWSLETTER_SUBSCRIBE = 'thelia.newsletter.subscribe';
-
-    /**
-     * sent for subscribing to the newsletter
-     */
+    const NEWSLETTER_UPDATE = 'thelia.newsletter.update';
    const NEWSLETTER_UNSUBSCRIBE = 'thelia.newsletter.unsubscribe';
 }
--- a/core/lib/Thelia/Model/NewsletterQuery.php
+++ b/core/lib/Thelia/Model/NewsletterQuery.php
@@ -17,8 +17,5 @@ use Thelia\Model\Base\NewsletterQuery as BaseNewsletterQuery;
 */
 class NewsletterQuery extends BaseNewsletterQuery
 {
-    public static function isSubscribed($email)
-    {
-        return (null === self::create()->findOneByEmail($email)) ? false : true;
-    }
+
 } // NewsletterQuery
--- a/local/config/schema.xml
+++ b/local/config/schema.xml
@@ -84,7 +84,8 @@
    <column name="description" type="CLOB" />
    <column name="chapo" type="LONGVARCHAR" />
    <column name="postscriptum" type="LONGVARCHAR" />
-    <column name="by_default" type="TINYINT" />
+    <column defaultValue="0" name="by_default" type="TINYINT" />
+    <column defaultValue="0" name="shop_country" required="true" type="BOOLEAN" />
    <foreign-key foreignTable="area" name="fk_country_area_id" onDelete="SET NULL" onUpdate="RESTRICT">
      <reference foreign="id" local="area_id" />
    </foreign-key>
@@ -767,13 +768,12 @@
    <foreign-key foreignTable="profile" name="fk_admin_profile_id" onDelete="RESTRICT" onUpdate="RESTRICT">
      <reference foreign="id" local="profile_id" />
    </foreign-key>
-    <index name="fk_admin_profile_id">
+    <index name="idx_admin_profile_id">
      <index-column name="profile_id" />
    </index>
    <behavior name="timestampable" />
  </table>
  <table isCrossRef="true" name="profile_resource" namespace="Thelia\Model">
-    <column autoIncrement="true" name="id" primaryKey="true" required="true" type="INTEGER" />
    <column name="profile_id" primaryKey="true" required="true" type="INTEGER" />
    <column name="resource_id" primaryKey="true" required="true" type="INTEGER" />
    <column defaultValue="0" name="access" required="true" type="INTEGER" />
@@ -783,7 +783,7 @@
    <foreign-key foreignTable="resource" name="fk_profile_resource_resource_id" onDelete="CASCADE" onUpdate="RESTRICT">
      <reference foreign="id" local="resource_id" />
    </foreign-key>
-    <index name="id_profile_resource_profile_id">
+    <index name="idx_profile_resource_profile_id">
      <index-column name="profile_id" />
    </index>
    <index name="idx_profile_resource_resource_id">
@@ -792,9 +792,8 @@
    <behavior name="timestampable" />
  </table>
  <table name="profile_module" namespace="Thelia\Model">
-    <column autoIncrement="true" name="id" primaryKey="true" required="true" type="INTEGER" />
-    <column name="profile_id" required="true" type="INTEGER" />
-    <column name="module_id" type="INTEGER" />
+    <column name="profile_id" primaryKey="true" required="true" type="INTEGER" />
+    <column name="module_id" primaryKey="true" required="true" type="INTEGER" />
    <column defaultValue="0" name="access" type="TINYINT" />
    <foreign-key foreignTable="profile" name="fk_profile_module_profile_id" onDelete="CASCADE" onUpdate="CASCADE">
      <reference foreign="id" local="profile_id" />
@@ -932,16 +931,16 @@
    <column name="address_invoice_id" type="INTEGER" />
    <column name="currency_id" type="INTEGER" />
    <column defaultValue="0" name="discount" type="FLOAT" />
-    <foreign-key foreignTable="customer" name="fk_cart_customer_id">
+    <foreign-key foreignTable="customer" name="fk_cart_customer_id" onDelete="CASCADE" onUpdate="RESTRICT">
      <reference foreign="id" local="customer_id" />
    </foreign-key>
-    <foreign-key foreignTable="address" name="fk_cart_address_delivery_id">
+    <foreign-key foreignTable="address" name="fk_cart_address_delivery_id" onDelete="RESTRICT" onUpdate="RESTRICT">
      <reference foreign="id" local="address_delivery_id" />
    </foreign-key>
-    <foreign-key foreignTable="address" name="fk_cart_address_invoice_id">
+    <foreign-key foreignTable="address" name="fk_cart_address_invoice_id" onDelete="RESTRICT" onUpdate="RESTRICT">
      <reference foreign="id" local="address_invoice_id" />
    </foreign-key>
-    <foreign-key foreignTable="currency" name="fk_cart_currency_id">
+    <foreign-key foreignTable="currency" name="fk_cart_currency_id" onDelete="CASCADE" onUpdate="RESTRICT">
      <reference foreign="id" local="currency_id" />
    </foreign-key>
    <index name="idx_cart_customer_id">
@@ -972,13 +971,13 @@
    <column name="price_end_of_life" type="TIMESTAMP" />
    <column defaultValue="0" name="discount" type="FLOAT" />
    <column name="promo" type="INTEGER" />
-    <foreign-key foreignTable="cart" name="fk_cart_item_cart_id">
+    <foreign-key foreignTable="cart" name="fk_cart_item_cart_id" onDelete="CASCADE" onUpdate="RESTRICT">
      <reference foreign="id" local="cart_id" />
    </foreign-key>
-    <foreign-key foreignTable="product" name="fk_cart_item_product_id">
+    <foreign-key foreignTable="product" name="fk_cart_item_product_id" onDelete="CASCADE" onUpdate="RESTRICT">
      <reference foreign="id" local="product_id" />
    </foreign-key>
-    <foreign-key foreignTable="product_sale_elements" name="fk_cart_item_product_sale_elements_id">
+    <foreign-key foreignTable="product_sale_elements" name="fk_cart_item_product_sale_elements_id" onDelete="CASCADE" onUpdate="RESTRICT">
      <reference foreign="id" local="product_sale_elements_id" />
    </foreign-key>
    <index name="idx_cart_item_cart_id">
@@ -995,8 +994,9 @@
  <table name="product_price" namespace="Thelia\Model">
    <column name="product_sale_elements_id" primaryKey="true" required="true" type="INTEGER" />
    <column name="currency_id" primaryKey="true" required="true" type="INTEGER" />
-    <column name="price" required="true" type="FLOAT" />
-    <column name="promo_price" type="FLOAT" />
+    <column defaultValue="0" name="price" required="true" type="FLOAT" />
+    <column defaultValue="0" name="promo_price" required="true" type="FLOAT" />
+    <column defaultValue="1" name="from_default_currency" required="true" type="BOOLEAN" />
    <foreign-key foreignTable="product_sale_elements" name="fk_product_price_product_sale_elements_id" onDelete="CASCADE">
      <reference foreign="id" local="product_sale_elements_id" />
    </foreign-key>
@@ -1250,7 +1250,10 @@
    <column name="email" required="true" size="255" type="VARCHAR" />
    <column name="firstname" size="255" type="VARCHAR" />
    <column name="lastname" size="255" type="VARCHAR" />
-    <column name="locale" size="45" type="VARCHAR" />
+    <column name="locale" size="5" type="VARCHAR" />
+    <unique name="email_UNIQUE">
+      <unique-column name="email" />
+    </unique>
    <behavior name="timestampable" />
  </table>
 </database>