Skip to content
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

Blog Entry pub_time default is wrong #10

Open
finkedave opened this issue Dec 6, 2012 · 0 comments
Open

Blog Entry pub_time default is wrong #10

finkedave opened this issue Dec 6, 2012 · 0 comments

Comments

@finkedave
Copy link
Contributor

I introduced this bug but since I restarted my server during development after every edit I never saw the issue.

Currently the default for Entry pub_time is
default=datetime.datetime.now().time

which is wrong. since now() is being called during instantiation. And the model uses that time for all new instances created.The time is over the place because on apache processes are being created and destroyed

This line should be
default=datetime.datetime.now

As long as the default is a function, it will be called for every new instance.

This is is a pretty problematic bug on the Times now. Sorry for introducing it. Its a one line change, let me know if you need me to make the change and do a pull request.

thanks
Dave

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant