Return-path: Envelope-to: miguel@ssf.com.mx Delivery-date: Wed, 06 May 2015 22:10:57 -0500 Received: from lb01-ash.bitbucket.org ([131.103.20.165]:15738 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 1YqCDA-000U2c-Vb for miguel@ssf.com.mx; Wed, 06 May 2015 22:10:57 -0500 Received: from app09.ash-private.bitbucket.org (app09.ash-private.bitbucket.org [172.17.10.21]) by lb01.bitbucket.org (Postfix) with ESMTP id 55A49A8291A for ; Thu, 7 May 2015 03:10:58 +0000 (UTC) Received: from app09.ash-private.bitbucket.org (localhost [127.0.0.1]) by app09.ash-private.bitbucket.org (Postfix) with ESMTP id 552363A015D for ; Thu, 7 May 2015 03:10:58 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8751110011457674197==" 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:10:58 -0000 Auto-Submitted: auto-generated Message-ID: References:

Precedence: bulk In-Reply-To: --===============8751110011457674197== 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-6707177 Ricardo Sansores said: Veo que eliminaste el JAVADOC. Hay que colocarlo de nuevo. No solo lo que ya estaba, sino añadiendo una descripción clara de para que sirve el método. Simplemente viendo la la firma no me sé para que sirve el método, me imagino que es para obtener la forma pero no está muy claro. ¿Porqué está en hardcoded null? -- Unwatch this pull request to stop receiving email updates: https://bitbucket.org/globalgassf/globalnet/pull-request/201/unwatch/miguelssf/227040c70998db5fb99dd488deb32b6be69c7311/ --===============8751110011457674197== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 8bit

app/controllers/RequisitionController.java

-     * @return Contenido de la vista.

-     */

     @Override

     protected Content getContent(Form<Requisition> form) {

Ricardo Sansores commented on pull request #201:

Requisicion y Detalle de requisicion

Veo que eliminaste el JAVADOC. Hay que colocarlo de nuevo. No solo lo que ya estaba, sino añadiendo una descripción clara de para que sirve el método. Simplemente viendo la la firma no me sé para que sirve el método, me imagino que es para obtener la forma pero no está muy claro. ¿Porqué está en hardcoded null?

View this pull request or add a comment by replying to this email.

-        return views.html.accountingModule.requisition.render(form);

+        return null;

}

Unwatch this pull request to stop receiving email updates.

--===============8751110011457674197==--