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 autoimport database wrapper #502

Merged
merged 35 commits into from Jul 29, 2022
Merged

Add autoimport database wrapper #502

merged 35 commits into from Jul 29, 2022

Conversation

lieryan
Copy link
Member

@lieryan lieryan commented Jul 27, 2022 •
edited

Description

Add a minimal database wrapper and query generator. This is to avoid doing string manipulations on SQL code, enforcing use of parameterized query.

Checklist (delete if not relevant):

  • I have added tests that prove my fix is effective or that my feature works
  • I have updated CHANGELOG.md

@lieryan
Move table creation to models.py
eeac655
@lieryan
Use concat operator to make the sql a constant
cc5a3bf
@lieryan
Move SQL to Name.search_submodule_like
297f5c9
@lieryan
Move SQL to Name.search_module_like
aeec2d2
@lieryan
Move SQL to Name.import_assist
1bc43b0
@lieryan
Use upper case consistently when writing SQL
119d2a2
@lieryan
Move SQL to Package.insert
9b85a57
@lieryan
Move SQL to Name.insert
040dc95
@lieryan
Move SQL to Name.search_name_like
5b97dc2
@lieryan
Move SQL to Name.get_modules
414d55e
@lieryan
Move SQL to Name.get_all_names
5cf2f11
@lieryan
Move SQL to Name.select_all and Package.select_all
b7fd1b0
@lieryan
Move SQL to Package.delete_by_package_name
4eedd5c
@lieryan
Move SQL to Name.drop_table and Package.drop_table
fab0fe4
@lieryan
Move SQL to Name.delete_by_module_name
b713019
@lieryan
Move SQL to Name.get_name_locations
52e0bc7
@lieryan
Implement basic Query wrapper
72f215a
@lieryan
Merge similar SQL into Name.search_by_name_like()
2d9dcf5
@lieryan
Merge similar SQL into Name.get_all
d7e7665
@lieryan
Implement Query.where(), move .select() to sqlite.py
41e8510
@lieryan
Change to Package.get_all
4360a5b
@lieryan
Automate creation of insert_into and drop_table
1787c7b
@lieryan
Refactor get_all to use table_name
c702145
@lieryan
Implement FinalQuery
4d43235
@lieryan
Remove dynamic constructs that are hard for mypy to typecheck
adef48f
@lieryan
Rename get_all->objects
9e99cc8
@lieryan
Implement _executemany()
76864c6
@lieryan
Implement delete_from()
c28c538
@lieryan
Type hint models.py
cbed84c
@lieryan
Blacken
dd51fb7
@lieryan lieryan changed the title Lieryan autoimport database wrapper Add autoimport database wrapper Jul 29, 2022
@lieryan lieryan merged commit 654e062 into master Jul 29, 2022
@lieryan lieryan deleted the lieryan-autoimport-database-wrapper branch July 29, 2022 12:42
@lieryan lieryan added this to the 1.3.0 milestone Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.0
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@lieryan