🇸🇬Singapore @vhin0210

vhin0210 → created an issue.

Noticed that the count is missing after the patch. Tweaked #4 ✨ Meta should include information about the view Needs work a little bit.

vhin0210 → made their first commit to this issue’s fork.

Interesting. I don't know exactly what happened but it's something similar to this
https://www.drupal.org/project/unpublished_node_permissions/issues/3427115 🐛 SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'gid' cannot be null: INSERT INTO "node_access" Needs review

Is this related or the same with this:
https://www.drupal.org/project/bootstrap4_modal/issues/3457776 ✨ Update events triggers Active

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

My theory is that some author/users were deleted and that makes the node owner to be null (anonymous) ?

The code is missing from 2.0.x

vhin0210 → created an issue.

Deleting
advancedqueue.plugin_type.yml: plugin_configuration_schema_id

works for me.

vhin0210 → changed the visibility of the branch 8.x-1.x to hidden.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → changed the visibility of the branch 3.x to hidden.

vhin0210 → changed the visibility of the branch 3472098-add-entity-block to hidden.

vhin0210 → created an issue.

Found a workaround for this.

What I did is I override the commerce_invoice.invoice_confirmation_subscriber on my custom module and did this:

/**
   * {@inheritdoc}
   */
  public function sendInvoiceConfirmation(WorkflowTransitionEvent $event) {
    parent::sendInvoiceConfirmation($event);

    $invoice_storage = $this->entityTypeManager->getStorage('commerce_invoice');
    $invoice_type_storage = $this->entityTypeManager->getStorage('commerce_invoice_type');
    /** @var \Drupal\commerce_invoice\Entity\InvoiceInterface[] $invoices */
    $invoices = $invoice_storage->loadMultiple($this->invoicesList);
    foreach ($invoices as $invoice) {
      /** @var \Drupal\commerce_invoice\Entity\InvoiceTypeInterface $invoice_type */
      $invoice_type = $invoice_type_storage->load($invoice->bundle());
      $this->invoiceConfirmationMail->send($invoice, $invoice->getEmail(), $invoice_type->getConfirmationBcc());
    }
  }

  /**
   * {@inheritdoc}
   */
  public function destruct() {
    return;
  }

In summary, I moved the send mail logic from destruct() to inside sendInvoiceConfirmation().

Sending from destruct() somehow triggering the session error.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → created an issue.

Ooops MB, can check now?

vhin0210 → created an issue.

Updated patch #68 🐛 Unable to resolve path on node in other language than default Needs work

So this patch has all the fixes from

RouterPathTranslatorSubscriber uses wrong param when resolving the url #2 🐛 RouterPathTranslatorSubscriber uses wrong param when resolving the url Active
Unable to resolve path on path with fragments #2 🐛 Unable to resolve path on path with fragments Needs review

and #66 🐛 Unable to resolve path on node in other language than default Needs work

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → created an issue.

vhin0210 → made their first commit to this issue’s fork.

Added the patch from Unable to resolve path on path with fragments 🐛 Unable to resolve path on path with fragments Needs review in patch #66 🐛 Unable to resolve path on node in other language than default Needs work

vhin0210 → created an issue.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

vhin0210 → made their first commit to this issue’s fork.

Subscribed. Have the same issue as #9 🐛 mail attachment not included Active

Still having this issue. How do we fix this?

Subscribed. I have the same issue as #5 🐛 RuntimeException: Failed to start the session because headers have already been sent by Response.php Active

Tried a different approach. Please review.

It's working now. Thanks!

vhin0210 → created an issue.

Please checkout my latest push

vhin0210 → made their first commit to this issue’s fork.