sedovalx / taxi

6 stars 3 forks source link

Проверка правильности заполнения полей для водителей #47

Closed kirzas closed 9 years ago

kirzas commented 9 years ago

@sedovalx в рамках #44 ты уже исправлял "ошибки валидации", может тут другое, но на всякий случай ссылаюсь. Сейчас при создании водителя ФИО не считаются обязательными, т.е. с сервера не приходит ошибка (см. скриншот), но при этом если остальные поля заполнить, то водитель все равно не создается.

capture

sedovalx commented 9 years ago

Не создается с какой ошибкой?

kirzas commented 9 years ago

@sedovalx @hodkoff2 Немного не так все :) Создается без ФИО, надо эти поля сделать обязательными. Или Дима, скажи, должны ли поля быть обязательными.

Не создается, если запись с таким же номером паспорта уже есть. Это логически правильно, но пользователь ошибки не видит, а на сервере такая ошибка: play.api.Application$$anon$1: Execution exception[[PSQLException: ╬╪╚┴╩└: яютЄюЁ ■∙ххё  чэрўхэшх ъы■ўр эрЁє°рхЄ юуЁрэшўхэшх єэшъры№эюёЄш "idx_pass_uq" Detail: ╩ы■ў "(pass)=(345345)" єцх ёє∙хёЄтєхЄ.]] at play.api.Application$class.handleError(Application.scala:296) ~[play_2.11-2.3.8.jar:2.3.8] at play.api.DefaultApplication.handleError(Application.scala:402) [play_2.11-2.3.8.jar:2.3.8] at play.core.server.netty.PlayDefaultUpstreamHandler$$anonfun$3$$anonfun$applyOrElse$4.apply(PlayDefaultUpstreamHandler.scala:320) [play_2.11-2.3.8.jar:2.3.8] at play.core.server.netty.PlayDefaultUpstreamHandler$$anonfun$3$$anonfun$applyOrElse$4.apply(PlayDefaultUpstreamHandler.scala:320) [play_2.11-2.3.8.jar:2.3.8] at scala.Option.map(Option.scala:146) [scala-library-2.11.6.jar:na] Caused by: org.postgresql.util.PSQLException: ╬╪╚┴╩└: яютЄюЁ ■∙ххё  чэрўхэшх ъы■ўр эрЁє°рхЄ юуЁрэшўхэшх єэшъры№эюёЄш "idx_pass_uq" Detail: ╩ы■ў "(pass)=(345345)" єцх ёє∙хёЄтєхЄ. at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2270) ~[postgresql-9.4-1201-jdbc41.jar:9.4] at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:1998) ~[postgresql-9.4-1201-jdbc41.jar:9.4] at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:255) ~[postgresql-9.4-1201-jdbc41.jar:9.4] at org.postgresql.jdbc2.AbstractJdbc2Statement.execute(AbstractJdbc2Statement.java:570) ~[postgresql-9.4-1201-jdbc41.jar:9.4] at org.postgresql.jdbc2.AbstractJdbc2Statement.executeWithFlags(AbstractJdbc2Statement.java:420) ~[postgresql-9.4-1201-jdbc41.jar:9.4]

sedovalx commented 9 years ago

@hodkoff2 должны ли поля быть обязательными?

hodkovdd commented 9 years ago

@Argelein @sedovalx должны быть обязательными!

sedovalx commented 9 years ago

@hodkoff2 @Argelein отчество не должно быть обязательным, т.к. не у всех оно есть.