ifnul / ums-admission-frontend

Frontend part for UMS system
http://194.44.198.221
4 stars 11 forks source link

Персони: searching by "id персони" doesn't work and cause an error message to show #71

Open MPushkar opened 9 years ago

MPushkar commented 9 years ago

Steps to reproduce:

  1. Login to system under Admin and go to a "Персони" tab.
  2. Select "id персони" option in "Пошук по:" field.
  3. Type-in any valid id and click "OK" button.

Expected result: Only matched person should be shown within grid.

Actual result: Following error is shown: lnu_id_error

ivanursul commented 9 years ago

Можеш скинути який ріквест шлеться? там має бути шось типу /api/persons?id

MPushkar commented 9 years ago

Вот: http://194.44.198.222:8080/is-lnu-rest-api/api/persons?id=1&limit=25&offset=0

ivanursul commented 9 years ago

ну дивись, а який сенс в такому запиті ? http://194.44.198.222:8080/is-lnu-rest-api/api/persons?id=1&limit=25&offset=0

id=1 якщо такий запис існує, то тільки один, тому що id - ключ, ідентифікатор

limit=25 Якщо id може бути тільки з одним значенням, то який сенс вказувати скільки елементів потрібно дістати?

offset=0 Те саме

Замість такого методу можна використовувати ось такий метод /api/persons/1

Як на рахунок того, щоб ми цей випадок на бекенді відхендлили, заборонили фільтрувати суто по полю id ? Відповідно, фронтенд теж таку ситуацію передбачить

MPushkar commented 9 years ago

Я - только за, т.к. id всё равно является уникальным значением и нет смысла пытаться генерировать целую гирлянду результатов, если нам всё равно всегда будет возвращаться лишь один.

ivanursul commented 9 years ago

Ну ок, тоді ше придумаю як заборонити фільтрувати по id, і відпишусь

MPushkar commented 9 years ago

Update: Same issue exists for "серії ОС".

ivanursul commented 9 years ago

Update: Same issue exists for "серії ОС". Можеш кинути запит, який шлеться ?

ivanursul commented 9 years ago

серія ОС - це docSeries поле, відповідно я це також пофіксав.

MPushkar commented 9 years ago

Returned. Searching by "id" should be removed from a list of options for "Персони".