Error when creating new product

[flgatormike]

This library is great. I can get products, orders, customers, and even update a product, but when I try to create a product using CreateAsync, I get:
"Object reference not set to an instance of an object"
Any idea what the problem might be?
Thanks!

[nozzlegear]

Thanks for the bug report @flgatormike! This package has unit tests that run with every new release, including ones for creating a new product, and they don't seem to have this issue. Could you possibly post the full exception with stack trace so I can take a look? It might also help if you can post the code you're using to create a product.

[flgatormike]

Thanks for the reply! I am trying to create a product by getting a product and just changing the id and title and then creating the product. I have also tried manually building the product. The stack trace is below but it looks like it doesn't step into the relevant code. Any idea what I might be doing wrong?

            var product = await GetProduct(7550715560151);
            product.Id = 7550715560133;
            product.Title = "TEST";
            product = await service.CreateAsync(product);

at ShopifySharp.ShopifyService.<>c__DisplayClass27_01.<<ExecuteRequestAsync>b__0>d.MoveNext() at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
at ShopifySharp.DefaultRequestExecutionPolicy.d__01.MoveNext() at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
at ShopifySharp.ShopifyService.d__271.MoveNext() at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at ShopifySharp.ProductService.<CreateAsync>d__5.MoveNext() at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
at qb_shopify.Main.d__22.MoveNext() in C:_projects\emustiga\qb-shopify\Main.cs:line 368

[flgatormike]

I also just tried copying the code in the unit test and get the same exception:

    public async Task<Product> CreateTestProduct(ProductCreateOptions options = null)
    {
        var service = new ProductService(domain, accessToken);
        var obj = await service.CreateAsync(new Product()
        {
            Title = "Test Product",
            Vendor = "Test Vendor",
            BodyHtml = "<p>Test Product</p>",
            ProductType = "Product",
            Handle = Guid.NewGuid().ToString(),
            Images = new List<ProductImage>
            {
                new ProductImage
                {
                    Attachment = "R0lGODlhAQABAIAAAAAAAAAAACH5BAEAAAAALAAAAAABAAEAAAICRAEAOw=="
                }
            },
        }, options);

        return obj;
    }

[nozzlegear]

Hmm you're right, that stack trace doesn't really give us anything to go on. Let's take another approach: when ShopifySharp throws an exception, the exception will have a property on it named RawBody. This is the exact string that Shopify is replying with, it may contain an error message or at least give us something else to look into. Can you wrap your code in a try/catch and then log that RawBody property to post here?

try 
{
    product = await service.CreateAsync(product);
}
catch (ShopifySharp.ShopifyException ex)
{
	Console.WriteLine(ex.RawBody);
	throw;
}

[flgatormike]

Hmm, just tested and the ShopifyException isn't thrown, just the standard Exception.
Strange that I am able to list orders, products, customers, and even update a product, but only get this error with create.

[nozzlegear]

Ah that's interesting, so it's unlikely to be something related to Shopify's response itself unless it's simply not returning the expected JSON. I think our best bet at the moment is for you to clone this project and include it in your own project directly and see what happens when you debug it. That should at least point you to the source of the problem and hopefully I'll be able to write a fix for it.

Let me know if you need any help cloning ShopifySharp and including it in your project!

[nozzlegear]

Something else to try: I know you've copied the code from the test project and had it break there too, but in the other sample code you provided above it looks like you've got a method named GetProduct and you're using that base product data to create a new one. What happens if you use the productService.GetAsync instead of your custom method? Does that throw the same error when you try to create with it?

var product = await service.GetAsync(7550715560151);
product.Id = 7550715560133;
product.Title = "TEST";
product = await service.CreateAsync(product);

[flgatormike]

so here is where I am getting the error. rawResult looks like it is valid json, but data is null. It looks like the root element inside rawResult is "products" but rootElement is set to "product" (without the s) which looks like the cause of the error. Where would this be configured?

[nozzlegear]

Thanks! So it is the response json causing the problem. Could you post that raw result value here? I’ll run it in a unit test and see what happens.

…

-- Josh Harms

On Mon, Mar 14, 2022, at 17:12, flgatormike wrote: so here is where I am getting the error. rawResult looks like it is valid json, but I am getting an error on deserialization: image <https://user-images.githubusercontent.com/47644612/158269343-df36b5d2-edb5-46a5-852d-772b96d8038c.png> — Reply to this email directly, view it on GitHub <#723 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AASOE7HFWIK3AVNAJO7NSNLU762TTANCNFSM5P4DLBGA>. You are receiving this because you commented.Message ID: ***@***.***>

[flgatormike]

Yes, the problem in the json is the root element. The root of the json is "product" but the code is looking for "products" with an s.. not sure if this mismatch is a problem with the shopify store or my code?

[nozzlegear]

That would definitely be the source of the problem. The ProductService.CreateAsync method should be using the string "product" as the root. I'm unable to locate any point where it was using "products" in the git history, but I might just be overlooking it. Are you overriding the method at all with your own implementation? If not, have you tried updating to the latest version of ShopifySharp to see if that fixes the problem?

[flgatormike]

"products" (with the s) is coming back from shopify, so maybe this is a problem with the store configuration?

[flgatormike]

Below is the code I am using to create a test product, which makes a call to executerequestasync which returns the json below which is a list of products and that is where I am getting the error. Not sure why I am getting a list of products here, and that is where I am getting the error...

            var product = await service.CreateAsync(new Product()
            {
                Title = "Test Product",
                Vendor = "Test Vendor",
                BodyHtml = "<p>Test Product</p>",
                ProductType = "Product",
                Handle = Guid.NewGuid().ToString(),
                Images = new List<ProductImage>
                {
                    new ProductImage
                    {
                        Attachment = "R0lGODlhAQABAIAAAAAAAAAAACH5BAEAAAAALAAAAAABAAEAAAICRAEAOw=="
                    }
                },
            }, options);

{"products":[{"id":7553612677335,"title":"Case OPC-8301-3","body_html"]}}

[flgatormike]

When I get orders, I get a root element of "orders"
When I get products, I get a root element of "products"
When I get customers, I get a root element of "customers"
When I update a product, I get a root element of "product"
And those all work fine
But when I create a product, I get a root element of "products" when it should be "product"
Any ideas?