As for #2126485: Can we have a 2.x-dev release? "commerce_coupon_review_pane" is still referenced in checkout but never created, we should review all the checkout process views to see if there are more cases like this.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

torgosPizza’s picture

I believe I found the issue, and it's twofold:

1) I had loaded the new module code on top of the old, meaning there were still some leftover crufty Views defaults there.

2) The variable "commerce_coupon_review_pane_view" needs to be deleted from the database, because the old Coupon module had set this variable and the new 2.x still does reference it. This means an upgrade path will need to simply remove this variable and remove/disable the old pane view.

Everything else worked fine, though, after uninstalling 1.x and re-installing a clean 2.x.

pcambra’s picture

Thanks for the feedback @torgosPizza!

Checking this further I see that the checkout review pane view used is now called "order_coupon_list", I guess we should try to keep the name from 1.x to 2.x as there's no need to change it, I think the name was "commerce_coupon_review_pane"

roderik’s picture

Status: Active » Needs review
FileSize
2.07 KB

For completeness: I installed commerce_coupon 2.x 'new' (i.e. after completely uninstalling 1.x), and everything works fine. So it's an update issue.

The view name is not directly referenced, but through a variable. So that variable value should be updated. I guess it should only be updated if the "commerce_coupon_review_pane" is actually not present anymore, so that leads to the uploaded patch.

Note #1: I spotted that the update functions are incorrectly numbered (now that this is actually the 2.x branch). So included that in the same patch.

Note/question #2: Why are we actually renaming the view from 'commerce_coupon_review_pane' to 'order_coupon_list'?

If not, this new update isn't necessary. But obviously I only ask myself this question after coding it... :)

pcambra’s picture

Note/question #2: Why are we actually renaming the view from 'commerce_coupon_review_pane' to 'order_coupon_list'?

That's the point :), we shouldn't, IMHO the patch should be creating the view as 'commerce_coupon_review_pane' instead of 'order_coupon_list'

roderik’s picture

FileSize
3.1 KB
1.34 KB

Meh, crosspost. I'm too slow for this; was coding/testing a patch before commenting, but missed your conversation in the meantime :)

Here's a quick reroll that works in the site I'm tinkering with (after I manually changed the variable settings back). The .install patch that I picked up along the way, is split into a separate file.

roderik’s picture

FileSize
1.32 KB

Failed to change the first comment line in patch 5a.

pcambra’s picture

Thanks for the patch, -5 is the one right for this issue

For exploring upgrade paths, we've got https://drupal.org/node/2125383 better

dpolant’s picture

Status: Needs review » Closed (fixed)

This is pretty old and I am reasonably sure the problem has been fixed. I am going to close for now.