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

CreateDate and UpdateDate always have DateMinvalue #2137

Closed
Vrijdagonline opened this Issue Mar 11, 2018 · 9 comments

Comments

3 participants
@Vrijdagonline
Contributor

Vrijdagonline commented Mar 11, 2018

Hi,

Product Sorting on createdate and updatedate is not possible, because these values always have a datetime.minvalue .

@chrno1209

This comment has been minimized.

chrno1209 commented Oct 17, 2018

I have the same issue, do you have any workaround?

@Vrijdagonline

This comment has been minimized.

Contributor

Vrijdagonline commented Oct 17, 2018

I skipped the implementation of this feature. So I can't help you with a workaround. I would like to know how to fix this too, so if you find something, please share.

@chrno1209

This comment has been minimized.

chrno1209 commented Oct 17, 2018

I am skipping this too, leaving the time to finish the project first, and then get back to fix this.
My first thought is to catch event when Merchello is saving product into db, then modify CreatedDate and UpdateDate, but not sure if it works or not.
Will keep you posted on this :).

@YodasMyDad

This comment has been minimized.

Contributor

YodasMyDad commented Oct 17, 2018

I would highly suspect this is an examine date conversion issue. As the createDate and the updateDate are stored in the database correctly.

@Vrijdagonline

This comment has been minimized.

Contributor

Vrijdagonline commented Oct 17, 2018

@YodasMyDad You are right. I came to the same conclusion back then. But couldn't find where the values gets lost exactly.

@YodasMyDad

This comment has been minimized.

Contributor

YodasMyDad commented Oct 17, 2018

In hindsight we should not have used Examine/Lucene. I think it's an extra layer of headache that isn't needed.

@YodasMyDad YodasMyDad added the Bug label Oct 17, 2018

@YodasMyDad YodasMyDad added this to To Do in Merchello 2.7.0 Oct 17, 2018

@YodasMyDad YodasMyDad moved this from To Do to In Progress in Merchello 2.7.0 Oct 18, 2018

@YodasMyDad

This comment has been minimized.

Contributor

YodasMyDad commented Oct 18, 2018

Just looking into this a bit more. Where exactly are you sorting? As there is no date sorting in the back office? And where are you seeing the minvale dates?

@YodasMyDad

This comment has been minimized.

Contributor

YodasMyDad commented Oct 18, 2018

This is now fixed in #2186

@YodasMyDad YodasMyDad closed this Oct 18, 2018

@YodasMyDad YodasMyDad moved this from In Progress to Done in Merchello 2.7.0 Oct 18, 2018

@Vrijdagonline

This comment has been minimized.

Contributor

Vrijdagonline commented Oct 18, 2018

Cool! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment