-
Notifications
You must be signed in to change notification settings - Fork 115
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
Sql server layer2 #158
Open
AlbyVin
wants to merge
21
commits into
gamondue:main
Choose a base branch
from
AlbyVin:SQLServerLayer2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Sql server layer2 #158
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Introduced DataLayer abstract class with derivation of SqLite and SQL server: SqLite_DataLayer and SqlServer_DataLayer.cs Nothing working..
Compiles, runs and accesses the database, with no visible faults. NO OTHER TESTS!. The Db in the treeview is currently not working. Made an abstract class DataLayer and a derived class SqLite_DataLayer, divided into the previous partial classes (DL_name of file>.cs), now renamed Lite_<previous file name>.cs Eliminated Common.dl. Now the DataLayer is defined and used only in BusinessLayer
…ptt divided into SQLite and SQL server classes. NOT WORKING. Serv_X.cs copied from Sqlite SqLite_DataLayer files. Serv_YearsAndPeriodsManagement.cs modified to adapt to Sql server. NUnit project with one test being configured (_Serv_YearsAndPeriodsManagement) (NOT WORKING). Renamed TreeMpttDb to TreeMpttDb_SqLite, added TreeMpttDb_SqlServer. Modified all accesses to DataLayer selecting the database with conditional compiling with SQL_SERVER symbol. Access to database temporarily excluded in frmMain with SQL_SERVER conditional compiling. Added NuGet packages for SQL server and NUnit.
…that was correct. It now runs the test.
# Conflicts: # DataLayer/SqLite/Lite_YearsAndPeriodsManagement.cs # DataLayer/SqlServer/Serv_SqlToCreateDatabase.cs # DataLayer/SqlServer/Serv_YearsAndPeriodsManagement.cs # NUnitTests/NUnitDbTests.csproj
…MpttDb_SqLite and TreeMpttDb_SqlServer, that now inherit from TreeMpttDb. Completed and fixed the instantiation and use of TreeMpttDB, in all programs. Fix in Lite_ClassManagement of the conversion to a date (was string), since gave error in import from csv.
Automatic detection of new files didn't work when two files in the same day where present in the folder. Fixed GetValidDateFromString to consider also the time. Previously it was considering only the dates.
…oose frmLessons: refactored UI to fit all screen fine frmStudent: UI refactoring and adjusting, fixes on datagrid mouse actions frmSetup: enabled call of frmClassesManagement WinForms: erased ininfluent code in frmGradesClassSummary
Binding of selected columns in mifrmLessons, dgwAllLessons
frmGradesSummary, frmGroups spostamento in BusinessLayer delle parti di codice opportune Base: get grade e aggiunta voticino * SaveMacroGrade, bugfix * metodi crud (da finire) --------- Co-authored-by: Sara Cappelli <SARA.CAPPELLI@local.ittcesena.it> Co-authored-by: Gabriele MONTI <gamondue@users.noreply.github.com>
aggiunti altri metodi di create, un delete --------- Co-authored-by: lorenzo.belletti <lorenzo.belletti@PC-0215> Co-authored-by: Gabriele MONTI <gamondue@users.noreply.github.com>
* prova commit * aggiunta metodo delete X Years and Period * tutto funzioannte --------- Co-authored-by: Gabriele MONTI <gamondue@users.noreply.github.com>
…Layer.cs. Refactorings. DataLayer.cs was UTF-16 and was considered a binary file, Re-saved as UTF-8. A little of refactorings: added method ExistsTable in DataLayer.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
tutti e 4 i test funzionanti