Account created on 6 December 2005, about 19 years ago
#

Merge Requests

More

Recent comments

🇵🇹Portugal introfini

I occasionally encounter this error on multilingual websites as well. It typically resolves with a cache clear via `drush cr` or by running `drush ev "drupal_flush_all_caches();"`.

🇵🇹Portugal introfini

This way, you can add a custom message or, more interestingly, a spinner or animations.

Please review...

🇵🇹Portugal introfini

introfini made their first commit to this issue’s fork.

🇵🇹Portugal introfini

I'm also encountering this issue with RC13 in Drupal 9.5. The patch from issue #11 successfully resolves it. Thank you!

🇵🇹Portugal introfini

#244 patch is working on my project. Thanks.

🇵🇹Portugal introfini

Please review the following reference: Add out-of-stock text message to select options in Add to Cart Form Needs review . While it may not be an exact match for your needs, it should effectively accomplish the task.

🇵🇹Portugal introfini

The merge request (MR) introduces this functionality.

To configure the desired message, follow these steps:

  1. Navigate to /admin/commerce/config/stock/enforcement/settings.
  2. Set the desired message for out-of-stock notifications.

With these settings updated, each product variation will now display an indication if it is out of stock.

🇵🇹Portugal introfini

introfini changed the visibility of the branch cherry-pick-84c6a86a to hidden.

🇵🇹Portugal introfini

introfini made their first commit to this issue’s fork.

🇵🇹Portugal introfini

After all, the issue was related to communication between two DDEV projects. I can confirm it is now working.

🇵🇹Portugal introfini

I'm experiencing the same issue with the "Add to Cart" form in Drupal Commerce. I attempted to resolve it using the Views Load More module ( https://www.drupal.org/project/views_load_more ), but the problem persists.

🇵🇹Portugal introfini

I mentioned that other modules face this issue. Since this module does not use tjackocnr/intl-tel-input, it can freely implement a widget based on my suggestion. What are your thoughts on this?

🇵🇹Portugal introfini

To assist others facing the same issue, ensure that the "Ignore Case" processor is not enabled on your index for the country field. The country codes are stored in uppercase, and the countryManager->getList() method returns keys in uppercase as well. Any conversion to lowercase will cause the functionality to break.

🇵🇹Portugal introfini

The solution is straightforward: just add a new address in the new shipping form and click the 'Recalculate shipping' button. This action will display the UPS shipping method with shipping rates.

🇵🇹Portugal introfini

It depends on this issue: https://www.drupal.org/project/commerce_ups/issues/3387353#comment-15575683 OAuth 2.0 support? (D10+) Fixed

🇵🇹Portugal introfini

@tBKoT Is it okay to commit this change to UPSSdk.php so we can enable translation of the service names? ( Allow the UPS service names to be translated Active )?

    public function getServiceName(string $service): string {
        return isset($this->serviceNames[$service]) ? t($this->serviceNames[$service]) : '';
    }
🇵🇹Portugal introfini

introfini changed the visibility of the branch 3444079-date-is-hardcode to hidden.

🇵🇹Portugal introfini

introfini changed the visibility of the branch 3444079-fix-date-format to active.

🇵🇹Portugal introfini

introfini changed the visibility of the branch 3444079-fix-date-format to active.

🇵🇹Portugal introfini

introfini changed the visibility of the branch 3444079-fix-date-format to hidden.

🇵🇹Portugal introfini

introfini changed the visibility of the branch 3444079-date-is-hardcode to active.

🇵🇹Portugal introfini

introfini changed the visibility of the branch 3444079-date-is-hardcode to hidden.

🇵🇹Portugal introfini

The development branch is currently behind the 8.x-1.1 release. Will do the MR for the 8.x-1.1 release instead.

🇵🇹Portugal introfini

The merge request introduces the following enhancements:

  1. Support for revisions of taxonomy terms.
  2. Addition of a setting for "Filter by Revision Translation Affected". I'm not certain if there are any side effects to disabling this setting, but I observed that revisions with 'revision_translation_affected' set to NULL were overlooked by the 'revisionsToKeep' method.
  3. Addition of a 'Configure' link for the module on the 'Extend' page (admin/modules).

Given the substantial nature of these changes, thorough testing and careful review are essential.

🇵🇹Portugal introfini

The merge request adds initial support for creating UPS Shipping Labels. It introduces basic functionality for generating these labels and includes a dependency on the Commerce Shipping Label API module to aid integration and manage label operations efficiently.

Can the module maintainers confirm if it's okay to add a dependency on the Commerce Shipping Label API module before I continue development?

🇵🇹Portugal introfini

For those looking to achieve this, I recommend using the suggestion by @longwave:

use CommerceGuys\Addressing\Repository\CountryRepository;

function convert_two_letter_to_three_letter_country_code($two_letter_code) {
    $countryRepository = new CountryRepository();
    // Get the country information based on the two-letter code.
    $country = $countryRepository->get($two_letter_code);

    if ($country) {
        // Return the three-letter country code.
        return $country->getThreeLetterCode();
    }
    return null;
}

🇵🇹Portugal introfini

With this merge request, it's possible to enable this feature by setting it in settings.php.

// Enable direct database serving of assets (CSS/JS) for read-only filesystem environments
$settings['optimized_assets_proxy_dump_from_database'] = TRUE;

Note: the fix for 📌 Upgrade to Drupal 10.1.x Needs review is already included.

🇵🇹Portugal introfini

The only necessary modification appears to be switching the override method from \Drupal\optimized_assets_proxy\AssetDumper::dump to \Drupal\optimized_assets_proxy\AssetDumper::dumpToUri. This is because \Drupal\Core\Asset\AssetDumper::dump now returns \Drupal\Core\Asset\AssetDumper::dumpToUri.

🇵🇹Portugal introfini

introfini made their first commit to this issue’s fork.

🇵🇹Portugal introfini

I've reviewed the patch; it's functioning well overall. However, there are some key points to address:

  1. To avoid errors, implement the solution found here: 🐛 Error: Call to a member function getLength() Needs review
  2. If utilizing any default packages, refer to this important note: Default UPS packages types need updating Active
  3. The issue with Negotiated Rates not working has been resolved; I've already committed the fix to the merge request.

Testing Standard Rates vs. Negotiated Rates:

🇵🇹Portugal introfini

introfini made their first commit to this issue’s fork.

🇵🇹Portugal introfini

The problem was that the PackageType is not set when calculating rates.

The module commerce_fedex already does that: https://git.drupalcode.org/project/commerce_fedex/-/blob/6c4155e25354526...

There's ongoing discussion relevant to this issue in the Commerce Shipping module: https://www.drupal.org/project/commerce_shipping/issues/2988710 .

I am escalating this to a critical priority due to its direct impact on halting the order checkout process, thereby preventing order completion.

🇵🇹Portugal introfini

introfini made their first commit to this issue’s fork.

🇵🇹Portugal introfini

Patch #14 works well. Thanks!

🇵🇹Portugal introfini

The patch works. Thanks!

Note: If you have this patch applied before this one doesn't apply: https://www.drupal.org/project/download/issues/3324861 🐛 Declaration of Drupal\download\Plugin\Field\FieldFormatter\DownloadLinkFormatter::create not compatible RTBC

🇵🇹Portugal introfini

I also believe that it should have been set to disabled as the default configuration. Meanwhile, you can add the following to the settings.php file:

# Disable Commerce Inbox
$settings['commerce_dashboard_show_toolbar_link'] = FALSE;
$settings['commerce_dashboard_fetch_inbox_messages'] = FALSE;
🇵🇹Portugal introfini

I figured out why it wasn't working: the problem was that my admin account didn't have the Administrator role set.

🇵🇹Portugal introfini

Hi,

The related Drupal Commerce issue: 📌 Show payment instructions in the order receipt Needs work

You need to create a Twig variable that contains payment instructions. This variable can subsequently be incorporated into your email template.

Below is an example of how you might define this variable within Twig. Please note that the exact implementation may vary depending on the method you utilize to dispatch emails:

function bloom_custom_preprocess_easy_email_commerce_order_payment_method_html(&$variables)
{
    /** @var Drupal\commerce_order\Entity\OrderInterface $order */
    $order = $variables["commerce_order"];
    if ($order->get('payment_gateway')->isEmpty()) {
        return;
    }

    /** @var \Drupal\commerce_payment\Entity\PaymentGatewayInterface $payment_gateway */
    $payment_gateway = $order->get('payment_gateway')->entity;
    /** @var \Drupal\commerce_payment\Plugin\Commerce\PaymentGateway\HasPaymentInstructionsInterface $payment_gateway_plugin */
    $payment_gateway_plugin = $payment_gateway->getPlugin();
    if ($payment_gateway_plugin instanceof HasPaymentInstructionsInterface) {
        $payment_storage = \Drupal::entityTypeManager()
            ->getStorage('commerce_payment');
        $payments = $payment_storage->loadMultipleByOrder($order);
        $payments = array_filter($payments, function ($payment) use ($payment_gateway) {
            return $payment->getPaymentGatewayId() == $payment_gateway->id();
        });
        $payment = reset($payments);
        if ($payment) {
            $variables['payment_instructions'] = $payment_gateway_plugin->buildPaymentInstructions($payment);
        }
    }

}

Once you have defined the variable, you can include it in your email template using {{ payment_instructions }}

🇵🇹Portugal introfini

Here's a way:

// Add the GA user_id to google_tag
function bloom_custom_page_attachments_alter(array &$attachments)
{
    $current_user = \Drupal::currentUser();
    if (isset($attachments["#attached"]["drupalSettings"]["gtag"]) && $current_user->isAuthenticated()) {
        $attachments["#attached"]["drupalSettings"]["gtag"]["additionalConfigInfo"]['user_id'] = 'drupal-user-id-' . $current_user->id();
    }
}

🇵🇹Portugal introfini

Updated for node_revision_delete 1.0.0-rc6 new try

🇵🇹Portugal introfini

Updated for node_revision_delete 1.0.0-rc6

🇵🇹Portugal introfini

In my case it was happening because I was creating a new email entity of non existing type.

Thanks for the patch.

🇵🇹Portugal introfini

This way you can override like this:

    public function __construct()
    {
        $this->delimiter = ';';
    }
🇵🇹Portugal introfini

I experienced the same problem. The issue, I believe, is that when you install the module, this is configured by default:

You need to change that for something with min-with.

Production build 0.71.5 2024