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

Manual date input expects wrong format #21

Closed
jowi24 opened this issue Dec 6, 2016 · 4 comments
Closed

Manual date input expects wrong format #21

jowi24 opened this issue Dec 6, 2016 · 4 comments
Labels
Milestone

Comments

@jowi24
Copy link

jowi24 commented Dec 6, 2016

Steps to reproduce

  1. Login NC user that has german language active
  2. Create task
  3. Click into the task begin or task end date input field. "dd.mm.yyyy" is shown.
  4. Enter, e.g., "01.03.2017" for March 1st 2017

Expected behaviour

Input of dd.mm.yyy formatted date should be possible, as indicated in step (3).

Actual behaviour

It is only possible to enter the date in form of "03/01/2017" and press Enter. After this, the date is transformed to "01.03.2017". However, this is not intuitive.

Server configuration

Operating system:
Arch Linux
Web server:
Nginx
Database:
Mysql
PHP version:
7.0
ownCloud version: (see ownCloud admin page)
10.0.1
Tasks version: (see ownCloud apps page)
0.9.3
Updated from an older ownCloud or fresh install:
Update from 10.0
Signing status (ownCloud 9.0 and above):

No errors have been found.

List of activated apps:

Enabled:
  - activity: 2.3.2
  - admin_audit: 1.0.0
  - bookmarks: 0.9.0
  - calendar: true
  - comments: 1.0.0
  - contacts: 1.5.1
  - dav: 1.0.1
  - external: true
  - federatedfilesharing: 1.0.1
  - federation: 1.0.1
  - files: 1.5.2
  - files_external: 1.0.2
  - files_pdfviewer: 0.8.1
  - files_sharing: 1.0.0
  - files_texteditor: 2.1
  - files_trashbin: 1.0.0
  - files_versions: 1.3.0
  - files_videoplayer: 0.9.8
  - firstrunwizard: 1.1
  - gallery: 15.0.0
  - notifications: 0.3.0
  - password_policy: 1.0.0
  - provisioning_api: 1.0.0
  - serverinfo: 1.1.1
  - survey_client: 0.1.5
  - systemtags: 1.0.2
  - tasks: true
  - templateeditor: 0.1
  - theming: 1.0.1
  - updatenotification: 1.0.1
  - workflowengine: 1.0.1
Disabled:
  - encryption
  - files_accesscontrol
  - files_automatedtagging
  - files_retention
  - twofactor_totp
  - user_external
  - user_ldap
  - user_saml

The content of config/config.php:


    "system": {
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "datadirectory": "\/srv\/http\/data\/",
        "dbtype": "mysql",
        "version": "9.1.1.5",
        "dbname": "nextcloud",
        "dbhost": "localhost",
        "dbtableprefix": "oc_",
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "logtimezone": "UTC",
        "installed": true,
        "logfile": "\/var\/log\/nginx\/nextcloud.log",
        "loglevel": 2,
        "theme": "",
        "maintenance": false,
        "skeletondirectory": "",
        "appstore.experimental.enabled": true
    }
}

Are you using external storage, if yes which one: local/smb/sftp/...
no
Are you using encryption: yes/no
no
Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...
no

Client configuration

Browser:
Chrome
Operating system:
Windows 7
CalDAV-clients:
n/a

@korelstar
Copy link
Member

I can confirm this bug.

@korelstar korelstar added the bug label Dec 6, 2016
@veronica-oldilocks
Copy link

Hello all! Wanting to sink my teeth into Nextcloud development with a single bug. I assume this bug is still present since the issue is still here. :/

After spending some time this evening going over the Developer Guidelines, I’m hoping to set aside some time in the next week to set up a VM and start working. If this bug is still available, I’d be happy to take it on. If anyone out there already did some work on it, and wants to fill me in, please do!

@raimund-schluessler
Copy link
Member

raimund-schluessler commented Nov 14, 2017

@thevkc Good to hear you want to work on that, you are very welcome! The bug is indeed still present. To be honest, the date format is quite a mess in the app. To get things right, we would likely have to integrate this with the (to be developed) locale setting of the nextcloud server, see: nextcloud/server#1781 and respective PR nextcloud/server#5623.

To get this working in the app, we would need to set the locale in moment.js and probably hand over the correct format to the date- and timepicker. Also, it might be necessary to adjust the stuff in https://github.com/nextcloud/tasks/blob/master/js/app/app.js#L69-L177, altough I am not quite sure about that.

@raimund-schluessler
Copy link
Member

Fixed with #194.

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

No branches or pull requests

4 participants