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

Finished Database working #12

Merged
merged 50 commits into from Feb 9, 2019
Merged

Finished Database working #12

merged 50 commits into from Feb 9, 2019

Conversation

Javinator9889
Copy link
Owner

Able to merge - tests needed

Those methods do not correspond entry ones
Entry file needs new coverage and analysis
Moved some methods to parent class as they are working the same way for all subclasses
"cidCategory" is not necessary to be in Entry child classes/tables, so it was removed
…r" for obtaining a new SQL script and included new method at IOManager for reading OSQL files
@Javinator9889 Javinator9889 self-assigned this Nov 5, 2018
@Javinator9889
Copy link
Owner Author

@Javinator9889
Copy link
Owner Author

@Javinator9889
Copy link
Owner Author

…into database

# Conflicts:
#	APP/.idea/caches/build_file_checksums.ser
#	APP/app/build.gradle
#	APP/app/src/main/java/javinator9889/securepass/io/database/DatabaseManager.java
#	APP/app/src/main/java/javinator9889/securepass/io/database/operations/CommonOperations.java
Minor corrections on "CommonOperations" class (the executor now finishes also when "finishConnection()" is called) and also inside "CategoryOperations" class. The new method allows the ability to remove the ID field inside the "Enum" classes (in a future, enumerations will be replaced with "StringDef" - see: https://android.jlelse.eu/android-performance-avoid-using-enum-on-android-326be0794dc3)
Finally, lots of conflict files were removed as they are not essential
In SQLite, if there is any FOREIGN KEY, it will not be necessarily a PRIMARY KEY, as it can violate constraints (NOT NULL) and at some cases there will be NULL primary keys and multiple (NON UNIQUE) FOREIGN KEYS marked as PRIMARY KEYS
@Javinator9889 Javinator9889 merged commit 746dcab into javadoc Feb 9, 2019
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.

None yet

1 participant