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

BugFix date-find can not select date from calendar. #174

Merged
merged 1 commit into from
Jan 3, 2022

Conversation

chunlinyao
Copy link
Member

When use date-find widget in qvt mode. Click date from popup calendar doesn't set date to text input.

@jonesde
Copy link
Member

jonesde commented Jan 1, 2022

It looks like the changes remove the prior or default values in the value attribute. How does that help with the issue described?

@chunlinyao
Copy link
Member Author

chunlinyao commented Jan 3, 2022

It looks like the changes remove the prior or default values in the value attribute. How does that help with the issue described?

If fieldsJsName does not exist it will use value attribute. value attribute only do unidirection binding, change date from datepicker will not update value in form fields. Change it to v-model will update form fileds.
Only date-find has this issue. date-time and date-period work fine.

@jonesde
Copy link
Member

jonesde commented Jan 3, 2022

Ah yes, thank you, I missed the addition of the v-model and value in the diff, and good catch as this was missed in the Quasar migration which included more strict VueJS behavior.

@jonesde jonesde merged commit ffaeef4 into moqui:master Jan 3, 2022
gagaboy added a commit to gagaboy/moqui-runtime that referenced this pull request Jan 31, 2022
* commit '05f8e8e55895495c1a8e79e34563a89ed159841d': (34 commits)
  On DataExport screen handle entityNames parameter that is a comma separated string which happens in qapps, fixes latest issue with DataExport under qapps/qvt
  Fixed the problem that moqui cannot be deployed as non-root webapp in Tomcat (moqui#190)
  On DataImport screen add support for creating missing records, with warnings about incomplete PKs because they can cause duplicate records; on a side note this screen is now a good way to test data for incomplete PKs
  Some improvements to Data Check functionality on the Data Import screen, shows check results in a table and allows selective update of field values; a first pass on this, another thing to handle that isn't there yet is adding missing records though that can be done with existing add missing functionality, just not selectively within a file
  On UserGroupDetail add new UserGroupPreference.groupPriority field to add/update group pref forms
  Added a new column for linking to auto screen. (moqui#188)
  On System/dashboard screen add a try/catch to handle errors getting ES cluster info, found this to break the whole page when ES is down
  MFA REST API and Login screen improvements (moqui#187)
  Update Swagger UI to 4.1.3 with JS/CSS files loaded from cdnjs instead of locally to avoid having files in the repos making it easier to update over time, along with cleaning up an old mess and a few CVE and WS security issues: moqui#176, moqui#177, moqui#178, moqui#179, moqui#180, moqui#181, moqui#182, moqui#183, moqui#184, moqui#185, moqui#186
  Add .whitesource configuration file (moqui#173)
  BugFix date-find can not select date from calendar. (moqui#174)
  Remove empty lines before <!doctype html>, now it become the first line. (moqui#175)
  In qvt render mode m-text-line VueJS component add reverseFillMask prop, in qvt text-line macro if field has the number validation class then set reverse-fill-mask
  In Login.xml specify in-map for call to reset#Password service so that other parameters cannot be specified as url parameters/etc
  Removed unused code from last commit to System/dashboard for Request Details dialog
  Add Request Details dialog to System/dashboard for inspecting headers, etc; change /status transition in webroot.xml to use ec.user.getClientIp() to use client ip header settings to avoid X-Forwarded-For spoof
  Misc improvements suggested by Arzang Kasiri: in WebrootVue.qvt.js m-display preserve whitespace in output by using the quasar q-input autogrow option; show the current date/time in system default time zone on System/dashboard; on AutoFind do a count query and only require parameters if table has more than 1 million records
  Add new targetPath field to EntitySyncDetail screen
  Use new getStatusMap(true) variation now support in ECFI in moqui-framework
  In DataExport screen add CSV support, in DataSnapshot screen add CSV and JSON support and commented fields for isoDateTime and tableColumnNames, currently commented because they are only supported for CSV and not XML or JSON yet
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants