address selection: use API points for Country and PostalCode + remove obsolet Address Controller

src/Bundle/ChillMainBundle/Controller/AddressController.php

@@ -1,63 +0,0 @@
-<?php
-
-namespace Chill\MainBundle\Controller;
-
-use Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter;
-use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
-use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\HttpFoundation\Exception\BadRequestException;
-use Symfony\Component\Routing\Annotation\Route;
-use Chill\MainBundle\Entity\Address;
-use Chill\MainBundle\Entity\AddressReference;
-
-/**
- * Class AddressController
- *
- * @package Chill\MainBundle\Controller
- */
-class AddressController extends AbstractController
-{
-
-
-    /**
-     * Get API Data for showing endpoint
-     *
-     * @Route(
-     *     "/{_locale}/main/api/1.0/address/{address_id}/show.{_format}",
-     *     name="chill_main_address_api_show"
-     * )
-     * @ParamConverter("address", options={"id": "address_id"})
-     */
-    public function showAddress(Address $address, $_format): Response
-    {
-        // TODO check ACL ?
-        switch ($_format) {
-            case 'json':
-                return $this->json($address);
-            default:
-                throw new BadRequestException('Unsupported format');
-        }
-    }
-
-
-    /**
-     * Get API Data for showing endpoint
-     *
-     * @Route(
-     *     "/{_locale}/main/api/1.0/address-reference/{address_reference_id}/show.{_format}",
-     *     name="chill_main_address_reference_api_show"
-     * )
-     * @ParamConverter("addressReference", options={"id": "address_reference_id"})
-     */
-    public function showAddressReference(AddressReference $addressReference, $_format): Response
-    {
-        // TODO check ACL ?
-        switch ($_format) {
-            case 'json':
-                return $this->json($addressReference);
-            default:
-                throw new BadRequestException('Unsupported format');
-        }
-
-    }
-}

src/Bundle/ChillMainBundle/Resources/public/vuejs/_api/AddAddress.js

@@ -1,13 +1,17 @@
 /*
 * Endpoint countries GET
-* TODO
+* method GET, get Country Object
+* @returns {Promise} a promise containing all Country object
 */
 const fetchCountries = () => {
    console.log('<<< fetching countries');
-   return [
+
-      {id: 1, name: 'France', countryCode: 'FR'},
+   const url = `/api/1.0/main/country.json`;
-      {id: 2, name: 'Belgium', countryCode: 'BE'}
+   return fetch(url)
-   ];
+      .then(response => {
+         if (response.ok) { return response.json(); }
+         throw Error('Error with request resource response');
+      });
 };
 
 /*
@@ -16,11 +20,13 @@ const fetchCountries = () => {
 */
 const fetchCities = (country) => {
    console.log('<<< fetching cities for', country);
-   return [
+   //TODO use country
-      {id: 1, name: 'Bruxelles', code: '1000', country: 'BE'},
+   const url = `/api/1.0/main/postal-code.json`;
-      {id: 2, name: 'Aisne', code: '85045', country: 'FR'},
+   return fetch(url)
-      {id: 3, name: 'Saint-Gervais', code: '85230', country: 'FR'}
+      .then(response => {
-   ];
+         if (response.ok) { return response.json(); }
+         throw Error('Error with request resource response');
+      });
 };
 
 /*

src/Bundle/ChillMainBundle/Resources/public/vuejs/_components/AddAddress.vue

@@ -167,25 +167,23 @@ export default {
       },
       getCountries() {
          console.log('getCountries');
-         this.address.loaded.countries = fetchCountries(); // à remplacer par
+         fetchCountries().then(countries => new Promise((resolve, reject) => {
-         //  fetchCountries().then(countries => new Promise((resolve, reject) => {
+             this.address.loaded.countries = countries.results;
-         //     this.address.loaded.countries = countries;
+             resolve()
-         //     resolve()
+          }))
-         //  }))
+          .catch((error) => {
-         //  .catch((error) => { 
+             this.errorMsg.push(error.message);
-         //     this.errorMsg.push(error.message); 
+          });
-         //  }); 
       },
       getCities(country) {
          console.log('getCities for', country.name);
-         this.address.loaded.cities = fetchCities(); // à remplacer par
+         fetchCities(country).then(cities => new Promise((resolve, reject) => {
-         //  fetchCities(country).then(cities => new Promise((resolve, reject) => {
+            this.address.loaded.cities = cities.results;
-         //     this.address.loaded.cities = cities;
+            resolve()
-         //     resolve()
+         }))
-         //  }))
+         .catch((error) => {
-         //  .catch((error) => { 
+            this.errorMsg.push(error.message);
-         //     this.errorMsg.push(error.message); 
+         });
-         //  }); 
       },
       getReferenceAddresses(city) {
          console.log('getReferenceAddresses for', city.name);

src/Bundle/ChillMainBundle/Resources/public/vuejs/_components/AddAddress/CountrySelection.vue

@@ -8,7 +8,7 @@
             v-bind:item="item"
             v-bind:key="item.id"
             v-bind:value="item">
-            {{ item.name }}
+            {{ item.name.fr }}
          </option>
       </select>
    </div>
@@ -21,7 +21,7 @@ export default {
    props: ['address', 'getCities'],
    computed: {
       countries() {
-         return this.address.loaded.countries; 
+         return this.address.loaded.countries; // TODO place Belgium and France first
       },
       selected: {
          set(value) {