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

Patch for code that is not following best practices in Magento development #46

Closed
wants to merge 4 commits into from
Closed

Patch for code that is not following best practices in Magento development #46

wants to merge 4 commits into from

Commits on Jul 19, 2012

  1. Fixed non reasonable usage of memory for collection, where possible t… 

    …o use more appropriate load() method of the model.
    @IvanChepurnyi
    IvanChepurnyi committed Jul 19, 2012
    Configuration menu
    Copy the full SHA
    b80a751 View commit details
    Browse the repository at this point in the history

  2. Added fix for loaded object check, to prevent viewing form if address… 

    … not found.
    @IvanChepurnyi
    IvanChepurnyi committed Jul 19, 2012
    Configuration menu
    Copy the full SHA
    cd665b1 View commit details
    Browse the repository at this point in the history

Commits on Jul 24, 2012

  1. Fixed core issue related to invalid totals sorting in the quote. The … 

    …problem is visible if you have additional custom totals that need to be placed in between core ones.
    @IvanChepurnyi
    IvanChepurnyi committed Jul 24, 2012
    Configuration menu
    Copy the full SHA
    ae10d80 View commit details
    Browse the repository at this point in the history

  2. ! Revert changes from app/code/core/Mage/Sales/Model/Config/Ordered.p… 

    …hp, to move them into separate branch
    @IvanChepurnyi
    IvanChepurnyi committed Jul 24, 2012
    Configuration menu
    Copy the full SHA
    48fdf7d View commit details
    Browse the repository at this point in the history