Skip to content

Edit Entries#307

Open
CTH-npitx wants to merge 56 commits intomainfrom
edit-entries
Open

Edit Entries#307
CTH-npitx wants to merge 56 commits intomainfrom
edit-entries

Conversation

@CTH-npitx
Copy link
Copy Markdown
Collaborator

Edit the entries

the text was in the wrong order, fixed
This shows the exception in the error output box
test if save fixed. it did not
There was no error, but it kept the invalid price (I like that)

Now testing a negative quantity
correct incorrecting
it kept the negative quantity. Neat!

Fixing that
So, tested the loading of invalid numbers (as in manual csv changes). When loading it to edit an error would definitly occur (so add error handling for that), but for reading, the following:

incorrect index; loaded fine
invalid price; loaded fine
invalid quantity; loaded fine
This tests for possible errors that could occur during reading file. Basically, tests an incompatible entry (still unsure what was broken), and 3 different edits (as in if you make numbers incorrect or invalid via manual CSV edits. There would be an error during entry editing, but during load... well; )

incorrect index; loaded fine
invalid price; loaded fine
invalid quantity; loaded fine
get the price, and if it's bellow the minimum set it to the minimum
Clears all the entry box contents, and deselects table
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.

1 participant