New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
increment_id column in sales_order table too short? #34521
Comments
Hi @hans-vereyken. Thank you for your report.
Make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:
For more details, review the Magento Contributor Assistant documentation. Add a comment to assign the issue: To learn more about issue processing workflow, refer to the Code Contributions.
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
Hi @engcom-Delta. Thank you for working on this issue.
|
Hi @hans-vereyken, In fact , I checked in Data base for the columns specified with respect to table names. Hence added the label 'Needs Update' |
It's the Phoenix Media Tradebyte Connector module (Phoenix_TradebyteConnector), but it's proprietary, so I can't provide you the code. |
@engcom-Delta I don't have access to this issue but I was the one who found this originally, could it be possible to remove "Issue: needs update" with the above answer as answer for your question |
✅ Jira issue https://jira.corp.magento.com/browse/AC-1778 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Delta. Thank you for verifying the issue. |
While testing a marketplace integration we stumbled upon the fact that the 3rd party extension is expecting the increment_id to be 50 characters. It tries to save an increment_id of > 32 characters into the sales_order column. It's an easy workaround, but it feels like an oversight in the Magento core code:
Looking at all increment_id columns and there length, only sales_order.increment_id has a length of 32, all the others have a length of 50:
tablename:increment_id-field-length
sales_order:32
sales_order_grid:50
sales_shipment:50
sales_shipment_grid:50
sales_invoice:50
sales_invoice_grid:50
sales_creditmemo:50
sales_creditmemo_grid:50
The difference between sales_order an sales_order_grid bothers me the most :-)
Also increment_id columns in other modules (eg: Customer) have a length of 50.
Is it an oversight or is there a good reason for the sales_order.increment_id to be only 32 characters in length? For me it's a bug.
Thx to https://github.com/vacla He found the bug.
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.
The text was updated successfully, but these errors were encountered: