search

farisca / SI2013Tim1

ETF Sarajevo - SI 2013 Tim1 - Sistem za radne naloge komunalnog preduzeća
0 stars 2 forks source link

Zaključavanje "praznog reda" u tabeli za pretragu radnih naloga #126

Closed aimamovic3 closed 10 years ago

aimamovic3 commented 10 years ago

Komponenta: jKP/interfejs za privilegovanog korisnika

Commit ID: fbcb788bb83922063c87f473143829faaea985a6

Opis problema: Moguće je odabrati u tabeli za pretragu naloga prazan red i onda kliknuti na dugme Zaključi pri čemu se baca izuzetak.

Koraci za reproduciranje problema:

  1. Pokrenuti desktop aplikaciju jkp
  2. Logovati se kao privilegovani korisnik
  3. Na početnom ekranu odabrati tab pretraga
  4. Odabrati prazan red u tabeli
  5. Kliknuti na dugme Zaključi

Očekivani rezultat:

Potrebno je da se ispisuje greška da je odabran prazan red u tabeli ili da se ne dodaju automatski prazni redovi u tabelu pa da se ne mogu ni izabrati.

Dobijeni rezultat:

Bacanje izuzetka http://pokit.org/get/?c5a832f6fecb74942eedc06a1a74bb53.jpg

dazinovic commented 10 years ago

Ne pomažeš ni sebi ni nama prijavljivanjem istih stvari po stoti put. Zamolio bih te da prestaneš. Ubuduće ću ovakve prijave zatvarati bez komentara.

aimamovic3 commented 10 years ago

I prvi put kad sam prijavio stavljeno je da je duplikat i da nije validan bug, a niko ga prije mene nije prijavio, povezao si ga pretragom što nije isti bug.

dazinovic commented 10 years ago

Nekima očigledno treba crtati. Pretraga nije implementirana u potpunosti i taj bug je prijavljen minimalno deset puta. To implicira da sve funkcionalnosti povezane sa pretragom neće ispravno raditi i da nema smisla prijavljivati bugove uzrokovane nefunkcionalnom pretragom. Kada se pretraga implementira biće pozatvarani svi prijavljeni problemi vezani za pretragu i priložen odgovarajući commit id, te će biti potrebno ponovo testirati sve funkcionalnosti vezane za pretragu.

aimamovic3 commented 10 years ago

Jedini kojim treba crtati ste vi, u ovom bugu se ni ne koristi pretraga, dovoljno je otvorit tab pretrage, ne moraju se ni pretražit korisnici, a iz tabele se može odabrati prazan red jer su automatski dodani i klikati na dugmad ispod tabele (Zaključi, Storniraj, Modifikuj, Detaljnije) pri čemu će biti bačen izuzetak. Ponavljam još jednom, bug nije vezan za pretragu nego za tabelu i dugmad ispod tabele.

HarisE commented 10 years ago

Admire to će biti riješeno sa time da će se kod pretrage kreirati tabela sa onoliko redova koliko ce biti rezultata pretrage, tako da neće biti više tu bug. I zamolio bih vas za malo strpljenja i bez svadje :D

aimamovic3 commented 10 years ago

Pa i prijavio sam u bugu da je greška što je popunjena sa praznim redovima, a to bi bila greška i da pretraga radi (a i radi pretraga po izvršiocima), ali dobro eto nek bude kako vi kažete, do mene je.

HarisE commented 10 years ago

Dupli #100