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

Cannot Change Receiving Date in Receiving Detailed Reports #2843

Closed
robbytriadi opened this issue May 31, 2020 · 8 comments
Closed

Cannot Change Receiving Date in Receiving Detailed Reports #2843

robbytriadi opened this issue May 31, 2020 · 8 comments
Assignees
Labels
Milestone

Comments

@robbytriadi
Copy link
Contributor

robbytriadi commented May 31, 2020

Background information

IMPORTANT: If you choose to ignore this issue report template, your issue will be closed as we cannot help without the requested information.

Please make sure you tick (add an x between the square brackets with no spaces) the following check boxes:

  • [v] Reporting an issue of an unmodified OSPOS installation
  • [v] Checked open and closed issues and no similar issue was already reported (please make sure you searched!)
  • [] Read README, WHATS_NEW, INSTALL.md and UPGRADE
  • [] Read the FAQ for any known install and/or upgrade gotchas (in specific PHP extensions installed)
  • [] Read the wiki
  • [] Executed any database upgrade scripts if an upgrade pre 3.0.0 (e.g. database/2.4_to_3.0.sql)
  • [] Aware the installation code is in bintray (see README), and GitHub master is for developers only and therefore not complete nor stable

Installation information

  • OSPOS version is: 3.3.1 - c786d4
  • OSPOS git commit hash is:
  • PHP version is: 7.1.33
  • MySQL or MariaDB version is: 5.5.5-10.2.31-MariaDB
  • OS and version is: Linux
  • WebServer is: LiteSpeed
  • Selected language is: English
  • (If applicable) Docker installation:
  • (If applicable) Installation package for the LAMP/LEMP stack is:

Bug

Please write your issue here. If a bug, please make sure to provide as much information as possible including configuration settings (e.g. Decimals set, Tax mode), language and steps to reproduce the bug.

Cannot change receiving date in receiving detailed report after choose the date. Cannot submit the change. The screenshot:

image

There is an error message "Fields in red are required" but there is no field name in red

Thank you

@odiea
Copy link
Collaborator

odiea commented May 31, 2020

Did you look at the Date on the Date picker? I don't think it can be changed. Should be a readonly input.

2020-05-31 12_09_04-Glamour School Antalya WeB Tasarım KaliteliPanelci com _ Powered by OSPOS 3 3 2

@odiea odiea added the bug label May 31, 2020
@robbytriadi
Copy link
Contributor Author

Did you look at the Date on the Date picker? I don't think it can be changed. Should be a readonly input.

If we change the date, the date format will be different with the previous.
Thank you

@jekkos
Copy link
Member

jekkos commented Jun 5, 2020

@robbytriadi So you think that is the reason you can't change it? The dateformat is incorrect after you select another value? Do you see any exception in the dev console?

@odiea
Copy link
Collaborator

odiea commented Jun 5, 2020

@jekkos Did you see the starting date on this datepicker. December 1899. What else is wrong with it. I replaced mine with the 1 from Expenses and had no issues after that. The following is the code from Receivings form.

<?php $this->load->view('partial/datepicker_locale'); ?>
    $('#datetime').datetimepicker(pickerconfig);

@jekkos
Copy link
Member

jekkos commented Aug 10, 2020

@odiea so you suggest to replace this picker with the one from expenses? Does that solve the 1899 issue? That one might be due to a wrong date format parse / evaluation

@odiea
Copy link
Collaborator

odiea commented Aug 10, 2020

Or change it to a good Date picker. The expenses one worked for me.

@jekkos jekkos self-assigned this Nov 24, 2020
@jekkos jekkos added this to the 3.3.3 milestone Nov 24, 2020
jekkos added a commit that referenced this issue Nov 24, 2020
@jekkos
Copy link
Member

jekkos commented Nov 24, 2020

Fix is in master now.

@jekkos jekkos closed this as completed Nov 24, 2020
@robbytriadi
Copy link
Contributor Author

@jekkos thank you :)

zulfiqarpk92 pushed a commit to zulfiqarpk92/opensourcepos that referenced this issue Dec 5, 2020
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

3 participants