Skip to content

Fix manual prune date range query#1

Open
erAck wants to merge 1 commit intoPiwigo:masterfrom
erAck:fix-query
Open

Fix manual prune date range query#1
erAck wants to merge 1 commit intoPiwigo:masterfrom
erAck:fix-query

Conversation

@erAck
Copy link

@erAck erAck commented Jan 24, 2021

Executing a manual prune with date range (not the same day) resulted
in an SQL query error with
AND date > "2021-01-13;
that should be
AND date > "2021-01-13"

Regression from

commit d8ceb81e49253be999330f90a0c870667f85e094
CommitDate: Fri Mar 16 14:15:45 2018 +0100

    "summarized" column in History table is no longer used since Piwigo
    2.9. Removed "summarized" from sql requests.

Executing a manual prune with date range (*not* the same day) resulted
in an SQL query error with
AND date > "2021-01-13;
that should be
AND date > "2021-01-13"

Regression from

    commit d8ceb81
    CommitDate: Fri Mar 16 14:15:45 2018 +0100

        "summarized" column in History table is no longer used since Piwigo
        2.9. Removed "summarized" from sql requests.
@erAck
Copy link
Author

erAck commented Jan 12, 2022

@MatthieuLP interested in merging this?

@erAck
Copy link
Author

erAck commented Jan 17, 2022

@ddtddt Seen that you did the latest change now, maybe you could merge this long pending fix?

@erAck
Copy link
Author

erAck commented Sep 27, 2022

@plegall @MatthieuLP @ddtddt
Please consider this tiny fix for an annoying bug. Thanks.

@erAck
Copy link
Author

erAck commented Jul 9, 2023

I really don't understand why this absolutely minimalist possible fix is still not merged after two and a half years now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant