Error when setting product tags during sync

[tristantbg]

Description

craft\shopify\elements\Product::setTags(): Argument #1 ($tags) must be of type array|string, null given, called in /app/vendor/yiisoft/yii2/base/Component.php on line 185

Steps to reproduce

1. Sync products

Additional info

Name	Version
PHP version	8.2.26
Linux	6.8.0-90-generic
MySQL	9.4.0
Craft Pro	5.9.17

[linear]

PT-3072

[AugustMiller]

Edit: Wrote this assuming you’re on the fresh 7.0 release! Let me know what version of the plugin you’re using and I can revise.

---

Hmm—haven't seen this before! We are currently counting on Shopify to send an array of tags.

Should be an easy fix. In the interim, do you mind running a query like this on the CLI?

php craft shopify/api/query 'query { products(first: 10) { edges { node { id title tags } } } }'

(If you know which product is failing, you can instead call product("gid://shopify/Product/123456789") { ... } so you're not having to page through results.)

I don't need the full response; just curious whether they’re actually returning null for a value that the docs explicitly state is “non-null.” 🙃

[nfourtythree]

Hi @tristantbg

As @AugustMiller mentioned this is a bit of a strange one because tags should never return null.

Along with the GraphQL query he mentioned could you send the full stack trace? This will give us an idea of whether the issue is coming from syncing the products from Shopify or if it retrieving the product element from the DB.

I have pushed up a fix for both versions 6 and 7 as a precautionary measure (even though they shouldn't be needed). These fixes will be out in releases shortly.

Let us know about the above questions, thanks!

[plcdnl]

Hello the same happens to me when i try to reset the data. I just updated to the version 7.0.1

/app $ php craft shopify/data/reset
Resetting Shopify plugin data will permanently delete all:

products
variants
sync history
from Craft. (Data in Shopify will not be affected.)

Do you wish to continue? (yes|no) [no]:yes

Removing Products ...Error: craft\shopify\elements\Product::setTags(): Argument #1 ($tags) must be of type array|string, null given, called in /app/vendor/yiisoft/yii2/base/Component.php on line 185

[plcdnl]

php craft shopify/api/query 'query { products(first: 10) { edges { node { id title tags } } } }'

Running query... done! (0.236228s)

❌ There was a problem with the query: No Shopify session available. Please check your credentials and re-authorize the application, if necessary.

Failed to run php craft shopify/api/query query { products(first: 10) { edges { node { id title tags } } } }: exit status 69

[nfourtythree]

Hi @plcdnl

I have just tried this on 7.0.1 and 6.1.4 and both times got the following:

php craft shopify/data/reset
Resetting Shopify plugin data will permanently delete all:
  > products
  > variants
  > sync history
from Craft. (Data in Shopify will not be affected.)

Do you wish to continue? (yes|no) [no]: yes
  > Removing Products ... done
  > Removing shopify data ... done
  > Removing sync history ... done

Finished.

Are you able to provide the full stack trace?

Have you authorised your Shopify app with your project?

Thanks

[tristantbg]

Hi again,
@plcdnl and I are working on the same project.

When resetting we get the same problem :

/app $ php craft shopify/data/reset
Resetting Shopify plugin data will permanently delete all:
  > products
  > variants
  > sync history
from Craft. (Data in Shopify will not be affected.)

Do you wish to continue? (yes|no) [no]:yes
  > Removing Products ...Error: craft\shopify\elements\Product::setTags(): Argument #1 ($tags) must be of type array|string, null given, called in /app/vendor/yiisoft/yii2/base/Component.php on line 185

Thanks !

[nfourtythree]

HI @tristantbg & @plcdnl

Are you able to send a database backup to support@craftcms.com referencing this issue.

That way we will be able to look into it further.

Thanks!

[tristantbg]

Sure ! I just did it.

[nfourtythree]

Hi @tristantbg & @plcdnl

Thank you for sending across the DB in support. I have sent over a reply in that thread.

I am going to close this issue for now as I think it seems specific to your project.

Thanks