Return-path: Envelope-to: miguel@ssf.com.mx Delivery-date: Wed, 06 May 2015 22:29:16 -0500 Received: from lb01-ash.bitbucket.org ([131.103.20.165]:14806 helo=lb01.bitbucket.org) by vishnu.hosting-mexico.net with esmtps (TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256) (Exim 4.85) (envelope-from ) id 1YqCUu-000W2P-ME for miguel@ssf.com.mx; Wed, 06 May 2015 22:29:16 -0500 Received: from app08.ash-private.bitbucket.org (app08.ash-private.bitbucket.org [172.17.10.19]) by lb01.bitbucket.org (Postfix) with ESMTP id 4437BA808EA for ; Thu, 7 May 2015 03:29:18 +0000 (UTC) Received: from app08.ash-private.bitbucket.org (localhost [127.0.0.1]) by app08.ash-private.bitbucket.org (Postfix) with ESMTP id 44E281A00B2 for ; Thu, 7 May 2015 03:29:18 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4084538969775118633==" MIME-Version: 1.0 Subject: Re: [Bitbucket] Pull request #201: Requisicion y Detalle de requisicion (globalgassf/globalnet) From: "Ricardo Sansores" To: miguel@ssf.com.mx Date: Thu, 07 May 2015 03:29:18 -0000 Auto-Submitted: auto-generated Message-ID: References: Precedence: bulk In-Reply-To: --===============4084538969775118633== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit --- you can reply above this line --- New comment on pull request: https://bitbucket.org/globalgassf/globalnet/pull-request/201/requisicion-y-detalle-de-requisicion#comment-6707349 Ricardo Sansores said: El método arroja una exception cuando no hay resultado. Me imagino que cuando la requisición no tiene detalles. En la línea 146 se usa el método y no se maneja dicho caso. Eso me lleva a dos suposiciones: 1. ¿Porqué marcar a éste método como que arroja dicha exception si ésta es una RuntimeException? 2. ¿Porqué no está manejado el posible caso de error en el lugar en que se usa el método? ¿Tus pruebas estuvieron completas? ¿Que ocurre si el editas una requisición que no tiene detalles? Si eso no es posible por el flujo de la información entonces convendría poner un assert o algo por el estilo. -- Unwatch this pull request to stop receiving email updates: https://bitbucket.org/globalgassf/globalnet/pull-request/201/unwatch/miguelssf/227040c70998db5fb99dd488deb32b6be69c7311/ --===============4084538969775118633== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 8bit
Ricardo Sansores commented on pull request #201: Requisicion y Detalle de requisicion
app/controllers/RequisitionDetailController.java
124 + return goToIndex();
125 + }
126 +
127 + private Form<RequisitionDetail> fillFormFromDatabase(String requisitionDetailId, Boolean isDetail)throws NoResultException {
rsansores
Ricardo Sansores commented on pull request #201:
Requisicion y Detalle de requisicion

El método arroja una exception cuando no hay resultado. Me imagino que cuando la requisición no tiene detalles. En la línea 146 se usa el método y no se maneja dicho caso. Eso me lleva a dos suposiciones:

  1. ¿Porqué marcar a éste método como que arroja dicha exception si ésta es una RuntimeException?
  2. ¿Porqué no está manejado el posible caso de error en el lugar en que se usa el método?

¿Tus pruebas estuvieron completas? ¿Que ocurre si el editas una requisición que no tiene detalles? Si eso no es posible por el flujo de la información entonces convendría poner un assert o algo por el estilo.
View this pull request or add a comment by replying to this email.
128 + RequisitionDetail requisitionDetail;
129 + if(isDetail) {
130 + requisitionDetail = globalService.getById(UUID.fromString(requisitionDetailId));
Unwatch this pull request to stop receiving email updates. Bitbucket
--===============4084538969775118633==--