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

Placeholder for signature do not consider chosen log date #16677

Open
Lineflyer opened this issue Feb 21, 2025 · 1 comment · May be fixed by #16691
Open

Placeholder for signature do not consider chosen log date #16677

Lineflyer opened this issue Feb 21, 2025 · 1 comment · May be fixed by #16691
Assignees
Labels
Bug Issues classified as a bug

Comments

@Lineflyer
Copy link
Member

Describe your problem!

From support mail

When using a log template referring to date/weekday it always uses the current date although I set the log date to a certain other date before inserting the template.

How to reproduce?

  • Create a custom log template e.g. including [date] or [dayofweek]
  • Start logging a cache
  • Set the log date to some past date
  • Use + icon and select to insert the previously created log template

Actual result after these steps?

The placeholders are expanded using the current day/date

Expected result after these steps?

The placeholders should consider the selected log date

Reproducible

Yes

c:geo Version

2025.02.15-NB and current production

System information


Additional Information

I wonder why no one else ever found this "bug"

@Lineflyer Lineflyer added Bug Issues classified as a bug Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility and removed Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility labels Feb 21, 2025
@ztNFny ztNFny self-assigned this Feb 27, 2025
@ztNFny ztNFny linked a pull request Feb 27, 2025 that will close this issue
@murggel
Copy link
Contributor

murggel commented Feb 27, 2025

According to the online-help (https://manual.cgeo.org/en/mainmenu/settings#logging) the current date when composing the log is used.

Image

Probably we can add a new template for Log-Date / Time / Date & Time, so we don't change the current behaviour

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues classified as a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants