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

F13 nova funkcionalnost pretrazivanje pri dodavanju knjige #48

Merged
merged 18 commits into from
Jun 15, 2024

Conversation

vlovric21
Copy link
Contributor

Što je napravljeno

Zaposlenik knjižnice sada pri dodavanju nove knjige na UcAddNewBook ekranu ne mora ručno prolaziti kroz combo box element za žanr i autora knjige već može pretražiti autora i žanr što filtrira combo box preko kojega odabire iste. Funkcionalnost je testirana ručno, jediničnim testovima i integracijskim testovima.

Kako radi

Koristi se _TextChanged ponašanje elementa TextBox u kojemu se prepoznaje kada zaposlenik unosi slova. Prvo se provjerava ima li unosa u slučaju da zaposlenik obriše sve što je unio, u tom slučaju se samo učitavaju svi autori ili svi žanrovi u ComboBox element. U slučaju da je prenesen znak ili znakovi, poziva se funkcija pripadajućeg servisa koja vraća sve zapise u pripadajućim tablicama koje sadrže uneseno. Vraćena lista se zatim koristi kao izvor podataka ComboBox elementu. Naravno, ovo se odvija svaki put kada se unese znak u TextBox. Primjer za žanr, ista je logika i kod autora:

image

Testovi

Napisani su jedinični testovi i integracijski testovi za funkcionalnost, svi su testirani i prolaze, nalaze se u mapi "F13" u pripadajućim projektima:

image

image

@vlovric21 vlovric21 added the enhancement New feature or request label Jun 14, 2024
@vlovric21 vlovric21 self-assigned this Jun 14, 2024
Copy link
Contributor

@dmatijani dmatijani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Male napomene ili prijedlozi, ali sve u svemu, dobro je i kvalitetno napisano.

@vlovric21 vlovric21 merged commit b609520 into dev Jun 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants