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

XoopsFormTextDateSelect null / empty #1163

Open
nitrix78 opened this issue Feb 17, 2022 · 6 comments
Open

XoopsFormTextDateSelect null / empty #1163

nitrix78 opened this issue Feb 17, 2022 · 6 comments

Comments

@nitrix78
Copy link

Hello, it's not possible to have a empty date in XoopsFormTextDateSelect
It miss.
If it's null or 0, don't put time() by default.

@GregMage
Copy link
Contributor

I don't think that's a good solution. To have an empty date I advise to do like this:

image

Then just manage the option in your code

@nitrix78
Copy link
Author

In some case, i need to have an empy datetime (in specific form, by example: a deleted date of a product).
It's a business need, it's the controler to choose if the default date is time(), not plateform to force it.
With XoopsFormTextDateSelect it's not possible.

It seem there is a parameter who can make it in a previous version of Xoops :
https://xoops.org/modules/newbb/viewtopic.php?topic_id=70482

@nitrix78
Copy link
Author

@GregMage
Copy link
Contributor

That's what I was going to suggest, create your own class for your needs. For compatibility reasons, it is not wise to modify the class XoopsFormDateSelect

@nitrix78
Copy link
Author

OK, but for future version, it will appreciable tu not have to do this easy

@geekwright
Copy link
Contributor

There are several date/time related issues in XOOPS that will be resolved in the next generation.

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

No branches or pull requests

3 participants