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

V5 DataStructure class #612

Open
wants to merge 8 commits into
base: v5
Choose a base branch
from

Conversation

dannyparsons
Copy link
Contributor

@volloholic @Patowhiz @Ivanluv for your review and discussion

New files

  • clsDataStructure.vb
  • clsDataStructureStation.vb
  • clsGlobalVariables.vb

clsDataStructure.vb

  • DoAction() and UpdateTable() methods added to make all changes in the database for an action and produce audit record (not yet linked to the data adaptor)
  • DoAddRecord(), DoEvent(), DoCorrectionToRecord(), DoDeleteRecord() methods added. These methods modify dtbUpdateTable and will be called based on actions in user controls.
  • TestUpdateTable() method added to test functionality of above methods by simulating modifying records of a DataTable.

TO DO

  • Link clsDataStructure to the data call class/data adaptor so it can write to the database
  • Decide how user controls will link with Do...() methods. What is the simplest way for user controls to pass values to the Do...() methods? (DataRows? Dictionarys? - methods are currently inconsistent)
  • Write method (in data call?) to work out next free ID value when adding new records to a table - suggest calculate maximum and increment.
  • Build up clsDataStructureStation and other specific child classes

@Patowhiz
Copy link
Collaborator

Patowhiz commented Apr 3, 2020

@dannyparsons do you have any pending changes?

I've gone through the data structure. I can see functions responsible for doing all the CRUD database operations except the READ.
Probably I'm missing something which function does the reading?
I can see we are defining the data to be read by setting, the table name, key fields, value fields, looking at these variables they are used in writing only.

…ial functions for Station, created Test class
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.

2 participants