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

Use addData for order/quote fixture #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

villekoykka
Copy link
Contributor

Added customer_email to Order.yml

Added customer_email to Order.yml
@joshporter
Copy link

Thanks @villekoykka

Missed the addData() in that builder. One question, why have you unset $attributes['payment_method']) and $attributes['shipping_method']) from the attributes array?

@villekoykka
Copy link
Contributor Author

Hi, the field 'payment_method' doesn't exist either in sales_flat_quote or sales_flat_order.

Only the 'shipping_method' field exists in sales_flat_order but the builder assigns the value anyway to the order.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants