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.

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

Added new attributes and some aliases to the {cart} substitution #1725

Merged
merged 1 commit into from Nov 26, 2015

Conversation

roadster31
Copy link
Contributor

This PR is add attributes to the {cart} substitution

  1. A new weight attribute is added, to get the cart total weight.
  2. A new total_price_without_discount attribute is added, to get the cart total amount without taxes, excluding discount.
  3. The following aliases of existing attributes are added, to provide a better english syntax, or a more accurate name :
    • product_count, alias of count_product
    • item_count, alias of count_item
    • total_price_with_discount alias of total_price
    • total_taxed_price_with_discount alias of total_taxed_price
    • contains_virtual_product alias of is_virtual
    • total_tax_amount alias of total_vat

@roadster31 roadster31 changed the title Added 'weight' attribute, and some aliases to the {cart} substitution Added new attributes and some aliases to the {cart} substitution Oct 9, 2015
@bibich
Copy link
Contributor

bibich commented Oct 13, 2015

Can you just squash your commits and I will merge your PR.
Thanks.

@roadster31
Copy link
Contributor Author

Done !

There is a related PR in the Thelia doc : thelia/thelia.github.io#172

@bibich
Copy link
Contributor

bibich commented Oct 16, 2015

Thanks

@gillesbourgeat gillesbourgeat added this to the 2.3.0-alpha1 milestone Nov 19, 2015
gillesbourgeat added a commit that referenced this pull request Nov 26, 2015
Added new attributes and some aliases to the {cart} substitution
@gillesbourgeat gillesbourgeat merged commit 26f5fcc into thelia:master Nov 26, 2015
@roadster31 roadster31 deleted the cart-subst branch January 7, 2016 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants