logo
Welcome to our new AbleCommerce forums. As a guest, you may view the information here. To post to this forum, you must have a registered account with us, either as a new user evaluating AbleCommerce or an existing user of the application. For all questions related to the older version of Gold and earlier, please go to AbleCommerce Gold forum. Please use your AbleCommerce username and password to Login. New Registrations are disabled.

Notification

Icon
Error
Dates on Low Inventory report are dd/MM/yyyy format
Options
Go to last post Go to first unread
Previous Topic Next Topic
judy at Web2Market  
#1 Posted : Wednesday, January 6, 2021 11:19:51 AM(UTC)
judy at Web2Market

Rank: Advanced Member

Groups: Developers
Joined: 11/7/2018(UTC)
Posts: 303

Thanks: 21 times
Was thanked: 5 time(s) in 5 post(s)
In the ReportController:
AvailablityDate = (product.AvailabilityDate == DateTime.MinValue) ? string.Empty : product.AvailabilityDate.ToString("dd/MM/yyyy"),

Everywhere else they are MM/dd/yyyy

I think most of your customers are in US so why not use the US format like the dates in all other locations I have seen.
Edit- uploading image
2021-01-06_12-29-46.png (12kb) downloaded 0 time(s).
Another screenshot- compare date with calendar.
2021-01-06_12-31-27.png (17kb) downloaded 0 time(s).

Edited by user Wednesday, January 6, 2021 11:32:17 AM(UTC)  | Reason: Not specified
Back to top

Wanna join the discussion?! Login to your AbleCommerce Forums forum account. New Registrations are disabled.
Back to top  
judy at Web2Market  
#2 Posted : Wednesday, January 6, 2021 11:37:58 AM(UTC)
judy at Web2Market

Rank: Advanced Member

Groups: Developers
Joined: 11/7/2018(UTC)
Posts: 303

Thanks: 21 times
Was thanked: 5 time(s) in 5 post(s)
OK, am I imagining things?
I changed the date on a product in the low inventory report and it set all the dates on the page to the same thing and changed the in stock figures. I have not made any changes to the code.
2021-01-06_12-33-42.png (72kb) downloaded 3 time(s).
2021-01-06_12-35-24.png (63kb) downloaded 3 time(s).
Back to top
shaharyartiwana25816656  
#3 Posted : Thursday, January 7, 2021 2:59:23 AM(UTC)
shaharyar

Rank: Advanced Member

Groups: Admin, Developers, Registered, HelpDesk, Authorized User
Joined: 10/5/2018(UTC)
Posts: 704

Thanks: 5 times
Was thanked: 113 time(s) in 112 post(s)
Hi Judy,

Thanks for pointing this out.

I'll report an issue.

Thanks.
Back to top
shaharyartiwana25816656  
#4 Posted : Thursday, January 7, 2021 3:02:21 AM(UTC)
shaharyar

Rank: Advanced Member

Groups: Admin, Developers, Registered, HelpDesk, Authorized User
Joined: 10/5/2018(UTC)
Posts: 704

Thanks: 5 times
Was thanked: 113 time(s) in 112 post(s)
Quote:
OK, am I imagining things?
I changed the date on a product in the low inventory report and it set all the dates on the page to the same thing and changed the in stock figures. I have not made any changes to the code.


This is an issue already registered and fixed. It will be fixed in the next release.

Thanks for your support.
Back to top
judy at Web2Market  
#5 Posted : Thursday, January 7, 2021 5:59:43 AM(UTC)
judy at Web2Market

Rank: Advanced Member

Groups: Developers
Joined: 11/7/2018(UTC)
Posts: 303

Thanks: 21 times
Was thanked: 5 time(s) in 5 post(s)
Thanks, but can you post the fix? I am working on a big site and they use that report extensively. They are purchasing source code in case they need to patch bugs before AC comes out with patches so their business can continue to function efficiently. This is a pretty big bug- it can mess up hundreds of products.
Back to top
shaharyartiwana25816656  
#6 Posted : Tuesday, January 12, 2021 8:35:22 PM(UTC)
shaharyar

Rank: Advanced Member

Groups: Admin, Developers, Registered, HelpDesk, Authorized User
Joined: 10/5/2018(UTC)
Posts: 704

Thanks: 5 times
Was thanked: 113 time(s) in 112 post(s)
Hi Judy,

You can follow these steps to add a fix for this issue.

1- Open file \Website\Areas\Admin\Controllers\ReportController.cs
2- Replace the file content with the content in the file attached ReportController.txt (167kb) downloaded 1 time(s).
3- Compile the code
4- Before going live with these changes, first apply these changes to your local setup and confirm the issue is fixed.
Back to top
thanks 1 user thanked shaharyar for this useful post.
judy at Web2Market on 1/12/2021(UTC)
judy at Web2Market  
#7 Posted : Tuesday, January 12, 2021 11:02:23 PM(UTC)
judy at Web2Market

Rank: Advanced Member

Groups: Developers
Joined: 11/7/2018(UTC)
Posts: 303

Thanks: 21 times
Was thanked: 5 time(s) in 5 post(s)
Thank you so much for the prompt fix!
Back to top
Users browsing this topic
Guest (3)
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

Privacy Policy | Powered by YAF.NET 2.2.4.14 | YAF.NET © 2003-2024, Yet Another Forum.NET
This page was generated in 0.152 seconds.