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

wxTimePickerCtrl for GRM #5530

Closed
vomikan opened this issue Jan 12, 2023 · 7 comments
Closed

wxTimePickerCtrl for GRM #5530

vomikan opened this issue Jan 12, 2023 · 7 comments
Assignees
Milestone

Comments

@vomikan
Copy link
Member

vomikan commented Jan 12, 2023

Revision for timing in reports.

image

The following syntax can be used in SQL queries:

select * from CHECKINGACCOUNT_V1 c
where c.LASTUPDATEDTIME >='&single_date'||'T'||'&single_time'
@vomikan vomikan self-assigned this Jan 12, 2023
@vomikan vomikan added this to the v1.6.3 milestone Jan 12, 2023
vomikan added a commit to vomikan/moneymanagerex that referenced this issue Jan 12, 2023
@vomikan
Copy link
Member Author

vomikan commented Jan 12, 2023

It should be noted here that the time stored in the UTC time zone is not very practical.

image

Another unpleasant moment is that the time in the Custom dialog is saved without taking into account the time zone.
image

image

vomikan added a commit that referenced this issue Jan 12, 2023
@vomikan vomikan added the fixed label Jan 12, 2023
@n-stein
Copy link
Contributor

n-stein commented Jan 13, 2023

You can just convert to UTC for the database read:

select * from CHECKINGACCOUNT_V1 c
where c.LASTUPDATEDTIME >= datetime('&single_date'||'T'||'&single_time', 'UTC');

@vomikan
Copy link
Member Author

vomikan commented Jan 13, 2023

You can just convert to UTC for the database read:

Thanks for pointing this out.

But it's about something else. Data is saved to the database without conversion to UTC. #5533

@vomikan
Copy link
Member Author

vomikan commented Jan 13, 2023

You can just convert to UTC for the database read:

select * from CHECKINGACCOUNT_V1 c
where c.LASTUPDATEDTIME >= datetime('&single_date'||'T'||'&single_time', 'UTC');

This leads me to wonder if it is necessary in this line .ToUTC() ?
value = time->GetValue().ToUTC().FormatISOTime();

@vomikan
Copy link
Member Author

vomikan commented Jan 13, 2023

Transformations have to be done in 2 stages.

select c.* from CHECKINGACCOUNT_V1 c
where c.LASTUPDATEDTIME >= strftime('%Y-%m-%dT%H:%M:%S', datetime('&single_date'||'T'||'&single_time', 'UTC')) 

vomikan added a commit to vomikan/moneymanagerex that referenced this issue Jan 13, 2023
vomikan added a commit that referenced this issue Jan 13, 2023
@vomikan
Copy link
Member Author

vomikan commented Jan 13, 2023

#5533

select c.* from CHECKINGACCOUNT_V1 c
where c.LASTUPDATEDTIME >= strftime('%Y-%m-%dT%H:%M:%S', datetime('&single_date'||'T'||'&single_time', 'UTC')) 

@vomikan vomikan removed the fixed label Jan 13, 2023
@vomikan
Copy link
Member Author

vomikan commented Jan 13, 2023

Invalid time value in the test mode of the editor.

image

vomikan added a commit to vomikan/moneymanagerex that referenced this issue Jan 13, 2023
vomikan added a commit that referenced this issue Jan 13, 2023
@vomikan vomikan added the fixed label Jan 13, 2023
@vomikan vomikan closed this as completed Jan 17, 2023
@vomikan vomikan mentioned this issue Jun 9, 2023
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

2 participants