Account created on 15 May 2012, almost 13 years ago
#

Merge Requests

More

Recent comments

🇨🇾Cyprus alex.bukach

Neither $status nor $payment_gateway are used in the function sop it is safe to simply remove them.

🇨🇾Cyprus alex.bukach

It seems to be a common Commerce issue, see Cannot remove conditions if you remove required field values first 🐛 Cannot remove conditions if you remove required field values first Fixed .

🇨🇾Cyprus alex.bukach

Agree @vidorado, the proper fix is to find the formatter(s) that return NULL and fix them.

🇨🇾Cyprus alex.bukach

@berdir, do you still think your solution in #8 will not work?

Why should not we store the result of isMultilingual() like this?

🇨🇾Cyprus alex.bukach

I checked how it happened that the active domain is reset in those two places. Here are respective commits: 32920b2f and ac6552da, and it looks like we should simply check whether a domain is previously set before resetting it. Please see MR !130 for the suggested change.

🇨🇾Cyprus alex.bukach

Good point, thanks @smustgrave, I have also created a MR!137 against 7.0.x. The patch in #27 works for both 6.x and 7.x.

🇨🇾Cyprus alex.bukach

alex.bukach changed the visibility of the branch 3396769-resource-not-found-7 to active.

🇨🇾Cyprus alex.bukach

alex.bukach changed the visibility of the branch 3396769-resource-not-found-7 to hidden.

🇨🇾Cyprus alex.bukach

The patch #12 does not work on pages with invalid route parameters, e.g. /node/[nid] for a non-existent node ID. Here's an updated patch (the PR !86 is updated respectively).

🇨🇾Cyprus alex.bukach

Thanks @jim_b, I faced the same issue, and the workaround worked for me.

🇨🇾Cyprus alex.bukach

@ptmkenny no, it does not look like you addressed in 2.x so it remains relevant. I rebased your MR against the head of 2.x.

🇨🇾Cyprus alex.bukach

As the patch #8 is not applicable anymore, I created a MR !11 with the same fix.

🇨🇾Cyprus alex.bukach

The MR pipeline fails due to unrelated issues.

🇨🇾Cyprus alex.bukach

I tried MR !15, and a webform submission (an AJAX request) is sent earlier than reCaptcha token is received and populated. It looks like AJAX form submissions are not prevented for AJAX forms.

🇨🇾Cyprus alex.bukach

In our case it was EU Cookie Compliance module that loaded the Leaflet module scripts without the dependency.

🇨🇾Cyprus alex.bukach

@hannakras I suggest you to have a look at https://www.drupal.org/project/countdown/issues/796110 🐛 Day is cached Needs review and the MR there. I ended up reworking the whole module.

🇨🇾Cyprus alex.bukach

Re-rolled the patch #29 against head of 8.x-4.x, and opened corresponding merge request !6.

🇨🇾Cyprus alex.bukach

I have reworked the module to use JavaScript for counting so that the blocks can now be safely cached.

🇨🇾Cyprus alex.bukach

It is still relevant.

🇨🇾Cyprus alex.bukach

I confirm the module does not work as expected, I can see that issue in both 7.x and 8.x versions. The JavaScript code expects the numbers be wrapped in 's, however they are not. Fixing it was a part of a @burt.lo's patch in #2. Here's a 8.x patch.

🇨🇾Cyprus alex.bukach

Patch #221 seems to work fine for me, however I face the issue mentioned in #236 too. Here's an updated patch.

🇨🇾Cyprus alex.bukach

Created MR!86, here's the respective patch.

🇨🇾Cyprus alex.bukach

Alex Bukach made their first commit to this issue’s fork.

🇨🇾Cyprus alex.bukach

@cchoe1 @Seth H, isn't it a duplicate of https://www.drupal.org/node/3266382 (where @cchoe1 provided a similar patch)?

Are you sure the issue happens only because a payment method is treated as reusable?

🇨🇾Cyprus alex.bukach

Re-rolled the patch against the head of 8.x-1.x and created respective MR!102.

🇨🇾Cyprus alex.bukach

Re-rolled patch #7 against the head of 8.x.1-x and created MR!101.

🇨🇾Cyprus alex.bukach

Alex Bukach changed the visibility of the branch 3266382-register-after-checkout to active.

🇨🇾Cyprus alex.bukach

Alex Bukach changed the visibility of the branch 3266382-register-after-checkout to hidden.

🇨🇾Cyprus alex.bukach

@DanielVeza I am still able to reproduce it on a fresh Drupal 11 setup. Here are the steps:

  • Navigate to a page /node/[nid]/layout?destination=[destination].
  • Click "Discard changes".
  • Get redirected to the [destination] page instead of /node/[nid]/layout/discard-changes

Here's an updated patch.

🇨🇾Cyprus alex.bukach

@jsacksick, at currently our project uses only fields declared Commerce API's, so we can rework the code to uninstall it. However I believe Commerce API should not break commerce shipping functionality anyway. I have found other scenarios when shipping address is empty, and the attached patch does not fix them. Therefore I dig further and came to another solution I posted in an issue for Commerce API 🐛 Empty shipping address Needs review . Therefore I am closing this issue.

🇨🇾Cyprus alex.bukach

I have checked that skipping saving profile for computed fields resolves the issue. However as all order profile fields declared by the module are computed fields, it's not clear why do we need to save the profile there anyway.

🇨🇾Cyprus alex.bukach

@jsacksick sorry again, it is Commerce API that provides the base field, we do use it as well, and I am checking whether we still need it :)

🇨🇾Cyprus alex.bukach

Sorry, my fault @jsacksick, I have updated the issue description.

I understand it might be not the best solution, however it's ShippingInformation pane that saves a shipping profile and then almost immediately saves a related order which in the case contains the shipping profile as well (though stale), and I cannot see any other way to hook into the process.

The same solution was suggested in the " OrderVersionMismatchException caused by ShippingInformation.php unnecessarily saving the order ", however there were able to find more elegant solution. Can you see another way to fix it in the case?

Production build 0.71.5 2024