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.

Sign up for GitHub

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

Jump to bottom

Add SQLite compatibility and fix SQL Full Group By issues (> MySQL 5.5 and MariaDB) #413

Merged
merged 60 commits into from
Jan 9, 2024
Merged

Add SQLite compatibility and fix SQL Full Group By issues (> MySQL 5.5 and MariaDB) #413

merged 60 commits into from
Jan 9, 2024

Conversation

bonny
Copy link
Owner

@bonny bonny commented Dec 30, 2023 •
edited
Loading

Fixes #405 #394 #411 #409 #397 #312.

Left to do before release:

  • Add changelog.
  • Write blog post.
  • Test on some more sites.
    • Tested on:
      • MariaDB 10.3.39 (simple-history.com)
      • MariaDB 10.5.13 (local docker)
      • MySQL 5.7.36 (local docker)
      • sqlite (local docker, using wp sqlite feature plugin)
      • Find one MySQL 5.5 too!
  • Release.

bonny added 30 commits November 18, 2023 21:51
@bonny
Tests: Add npm scripts to test sql query on different sql servers
87a9f47
@bonny
Tests: Remove commented out docker compose things
379b1b9
@bonny
Add some comments to log query class
c2af052
@bonny
Tests: Add more log query tests
95401d6
@bonny
Tests: Add more log query occasions tests
2f10689
@bonny
Remove mysql_get_server_info() because unknown function says editor
4d8598d
@bonny
Merge branch 'main' into bonny/issue409-db-issues
927d815
@bonny
Misc log query cleanup
9b3529d
@bonny
Log query: Use arrays to store where clauses + misc cleanup
44351a5
@bonny
Log query: Add more arguments to prepare function and misc cleanup
80eef6f
@bonny
Log query: Add arguments users, user, loggers, messages, and more, to…
64144e3 
… prepare function
@bonny
Log query: Limit fix for occassions
e19d390
@bonny
Log query: use %d for integers
85ae317
@bonny
Log query: Specify columns to get when adding context instead of gett…
cfb166b 
…ing all
@bonny
Log query: Misc cleanup
539a074
@bonny
Log query: Add query_occasions(), query_overview(), add_contexts_to_l…
df451b6 
…og_rows(), get_max_min_ids(), and cleanup
@bonny
IP addresses are now shown on occasions
be45ae1
@bonny
Remove unused "log_rows_raw" key from occassions api response
0100bf5
@bonny
Log query: Remove $_GET from cache key
7baef32
@bonny
Log query: Occassions cleanup
6bc6c35
@bonny
Log query: Add functions get_inner_where() and get_outer_where()
5cc562e
@bonny
Remove "rep" column from Log_Query response.
0224b52
@bonny
Remove column "occasionsIDType" from Log Query
8f4a4cb
@bonny
Log query: Misc cleanup
43db5dc
@bonny
Log query: New full group by query seems to work (lot's of debug!)
cb9f85c
@bonny
Remove debug
596eefb
@bonny
Log query fixes
c281c51
@bonny
Log query: Count using count(*)
188afa0
@bonny
Log query test: Disable mark as incomplete
4636a23
@bonny
Log query: Move default args to prepare_args function
739eab0
@bonny bonny marked this pull request as draft December 30, 2023 11:43
bonny added 18 commits December 31, 2023 20:32
@bonny
Typos
4ab73b4
@bonny
Add dropin that adds information about add-ons in the sidebar
670b1ea
@bonny
Typos
674b99d
@bonny
Add text to "purged_events" message with information that the number …
4109fd0 
…of days to keep the log can be changed
@bonny
Add text to the clear log settings text with info how to modify numbe…
01d1ddc 
…r of days to keep the log
@bonny
Check date filtering works
e2e5186
@bonny
Readme: Add note about add-ons to sponsor headline
5461eeb
@bonny
Log query: Add caching to query_overview_sqlite()
8c42b61
@bonny
Tests: Add tests for single event occasions
924c6bd
@bonny
Update composer packages
951c904
@bonny
Composer: Upgrade wp-cli
b00d045
@bonny
Fix phpstan
d38c23a
@bonny
Log query: Fail silently
d6ce006
@bonny
Correct add-on link in sidebar
f403dd8
@bonny
Load occasions using more specific link target
61711aa
@bonny
Update readme
ce413c6
@bonny
Prepare for add-ons info in occasions text
4a4ae86
@bonny
Update readme
005b20e
@bonny bonny self-assigned this Jan 9, 2024
@bonny bonny marked this pull request as ready for review January 9, 2024 18:55
@bonny bonny merged commit 98b9654 into main Jan 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bonny bonny

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Date filter does not work on MariaDB
1 participant
@bonny