Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix AddressController.php #34152

Closed
wants to merge 12 commits into from
Closed

Fix AddressController.php #34152

wants to merge 12 commits into from

Commits on Oct 3, 2023

  1. Fix AddressController.php 

    Duplicate call function. Address form filled second time. Fix bugs: 
#34080
and
#33991
    @metacreo
    metacreo committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    9fd4adf View commit details
    Browse the repository at this point in the history

Commits on Oct 5, 2023

  1. Small rework AddressController.php 

    Initialize address if an id exists and prevent fill form with default country second time. If this is only form request. Add param id_country for CustomerAddressForm class.
    @metacreo
    metacreo committed Oct 5, 2023
    Configuration menu
    Copy the full SHA
    6e84db7 View commit details
    Browse the repository at this point in the history

  2. Update AddressController.php 

    Optimisation
    @metacreo
    metacreo committed Oct 5, 2023
    Configuration menu
    Copy the full SHA
    78ab289 View commit details
    Browse the repository at this point in the history

Commits on Oct 6, 2023

  1. Eliminate double call of fillWith

    @metacreo
    metacreo committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    7c2136a View commit details
    Browse the repository at this point in the history

Commits on Oct 8, 2023

  1. Update AddressController.php 

    Optimisation. Remove extra call of Tools class.
    @metacreo
    metacreo committed Oct 8, 2023
    Configuration menu
    Copy the full SHA
    1bbb594 View commit details
    Browse the repository at this point in the history

  2. Update AddressController.php

    @metacreo
    metacreo committed Oct 8, 2023
    Configuration menu
    Copy the full SHA
    eecd0c9 View commit details
    Browse the repository at this point in the history

  3. Update AddressController.php 

    Do magic. No more need this. ;)
    @metacreo
    metacreo committed Oct 8, 2023
    Configuration menu
    Copy the full SHA
    a73eeec View commit details
    Browse the repository at this point in the history

  4. Update controllers/front/AddressController.php 

    ok. by moving fillWith afer if (Tools::isSubmit('submitAddress')) {
all seems to resolve ... so I suggest this

Co-authored-by: Krystian Podemski <kpodemski@users.noreply.github.com>
    @metacreo @kpodemski
    metacreo and kpodemski committed Oct 8, 2023
    Configuration menu
    Copy the full SHA
    41849c5 View commit details
    Browse the repository at this point in the history

  5. Update AddressController.php 

    Resolved
    @metacreo
    metacreo committed Oct 8, 2023
    Configuration menu
    Copy the full SHA
    241f434 View commit details
    Browse the repository at this point in the history

Commits on Jan 10, 2024

  1. Update AddressController.php 

    No need second 0 param in getValue, because (int).
    @metacreo
    metacreo committed Jan 10, 2024
    Configuration menu
    Copy the full SHA
    a5e8553 View commit details
    Browse the repository at this point in the history

  2. Fix AddressController.php 

    Fix mistake
    @metacreo
    metacreo committed Jan 10, 2024
    Configuration menu
    Copy the full SHA
    b2a3c4b View commit details
    Browse the repository at this point in the history

Commits on Jan 11, 2024

  1. Merge branch 'PrestaShop:develop' into patch-1

    @metacreo
    metacreo committed Jan 11, 2024
    Configuration menu
    Copy the full SHA
    ee962a0 View commit details
    Browse the repository at this point in the history