HotcakesCommerce / hotcakes-commerce-core

The core of the e-commerce part of the overall solution. This is an ecommerce shopping cart solution built on top of the DNN (DotNetNuke) CMS. Anyone can do commerce online now!
https://mmmcommerce.com
MIT License
69 stars 55 forks source link

Text Input Choice Field causes JSON Parse Error. #450

Open james7342 opened 1 year ago

james7342 commented 1 year ago

Describe the bug

Found this issue and am able to duplicate it with a Product Level and Shared Choice Text Input Box. (https://mmmcommerce.com/Forums/g/Posts/t/326/Shared-Choice-Text-Input-Field-BUG)

Software Versions

To Reproduce

Steps to reproduce the behavior:

  1. Create Choice item for Text Input - Shared or at Product Level
  2. Col. 1, Row 1, Size 50
  3. View Product and enter text in input box.
  4. In browser console note: JSON Parse Error
  5. Unable to add product to Cart.

If you remove the Text Input Size works as expected.

Expected behavior

No error and Product adds to Cart.

Actual behavior

Unable to add product to Cart.

Screenshots

JSON Error: screenshot_804

Error log

Note: Debug DLL's N/A

WillStrohl commented 1 year ago

Thank you for reporting this issue. Our team will attempt to reproduce the issue and report back here once we're done.

ismaelTartola commented 1 year ago

@WillStrohl @james7342 I was able to reproduce the bug reported in this issue. I followed the same steps to reproduce. Only point that I think we should add a extra step to add the Shared Choice Item to the product. How that's sound? @james7342. These are my steps to reproduce proposal:

  1. Create Choice item for Text Input - Shared or at Product Level.
  2. Col. 1, Row 1, Size 50.
  3. Add the Created Shared Choice to a Product.
  4. View Product and enter text in input box.
  5. In browser console note: JSON Parse Error.
  6. Unable to add product to Cart.

Regards,

james7342 commented 1 year ago

@ismaelTartola, Yes, I didn’t specially call out adding the shared item step. It effects both and my thought was if it was fixed at the product level it would probably address the shared aspect of it also. Both options would have to be verified I think before the issue could be closed.

WillStrohl commented 1 year ago

Thanks, @james7342 for helping us triage this. We'll try to fix it as soon as we can. :)