Return-path: <pullrequests-reply@bitbucket.org>
Envelope-to: miguel@ssf.com.mx
Delivery-date: Wed, 06 May 2015 22:50:30 -0500
Received: from lb01-ash.bitbucket.org ([131.103.20.165]:5012 helo=lb01.bitbucket.org)
	by vishnu.hosting-mexico.net with esmtps (TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256)
	(Exim 4.85)
	(envelope-from <pullrequests-reply@bitbucket.org>)
	id 1YqCpS-000YHL-Ma
	for miguel@ssf.com.mx; Wed, 06 May 2015 22:50:30 -0500
Received: from app02.ash-private.bitbucket.org (app02.ash-private.bitbucket.org [172.17.10.13])
	by lb01.bitbucket.org (Postfix) with ESMTP id 4D7AEA8283D
	for <miguel@ssf.com.mx>; Thu,  7 May 2015 03:50:32 +0000 (UTC)
Received: from app02.ash-private.bitbucket.org (localhost [127.0.0.1])
	by app02.ash-private.bitbucket.org (Postfix) with ESMTP id 4E61D2601A4
	for <miguel@ssf.com.mx>; Thu,  7 May 2015 03:50:32 +0000 (UTC)
Content-Type: multipart/alternative;
 boundary="===============6761295920308143537=="
MIME-Version: 1.0
Subject: Re: [Bitbucket] Pull request #201: Requisicion y Detalle de
 requisicion (globalgassf/globalnet)
From: "Ricardo Sansores" <pullrequests-reply@bitbucket.org>
To: miguel@ssf.com.mx
Date: Thu, 07 May 2015 03:50:32 -0000
Auto-Submitted: auto-generated
Message-ID: <prcomment-6707551@bitbucket.org>
References: <pr-globalgassf/globalnet/201@bitbucket.org>
 <pr-globalgassf/globalnet/201@bitbucket.org>
Precedence: bulk
In-Reply-To: <pr-globalgassf/globalnet/201@bitbucket.org>

--===============6761295920308143537==
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-6707551

Ricardo Sansores said:

Formatear el código, he encontrado muchos lugares en que hay espacios, saltos de línea y cosas de sobra. El IDE tiene shortcuts para formatear todo y dejarlo perfectamente.

--

Unwatch this pull request to stop receiving email updates:
https://bitbucket.org/globalgassf/globalnet/pull-request/201/unwatch/miguelssf/227040c70998db5fb99dd488deb32b6be69c7311/

--===============6761295920308143537==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 8bit

<!DOCTYPE HTML>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
    
  </head>
  <body style="font: 14px/1.4285714 Arial, sans-serif; color: #333;">
    <table style="width: 100%; border-collapse: collapse;">
      <tbody>
        <tr>
          <td style="background: #f5f5f5; padding: 10px 10px 0; font: 14px/1.4285714 Arial, sans-serif;">
            <table style="width: 100%; border-collapse: collapse;">
              <tbody>
                
                
                
                
                  
                     <tr><td id="main" class="comment-container no-box" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; background-color: #ffffff; border-radius: 5px">
    <div style="border: none; border-radius: 5px; padding: 0; background: #f5f5f5">
      <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="main-comment" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0">
              <div style="border: 1px solid #ccc; border-radius: 5px; background-color: #fff; padding: 0">
                
<span style="display: none; display: none !important;">
  
    Ricardo Sansores commented on pull request #201: Requisicion y Detalle de requisicion
  
</span>
<table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="comment-filename" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td style="font: 14px/1.4285714 Arial, sans-serif; padding: 10px; background: #f5f5f5; border-radius: 5px 5px 0 0">app/mx/ssf/globalgas/services/accounting/RequisitionDetailService.java</td>
              </tr></tbody></table></td>
      </tr><tr><td class="diff-line addition first" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px; border-top: 1px solid #ccc">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>88</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc'>+    public Requisit<wbr>ionDetail saveWithJs<wbr>on(JSONObject requis<wbr>itionDetail, String <wbr>userName, String fac<wbr>tory) throws JSONExc<wbr>eption {</wbr></wbr></wbr></wbr></wbr></wbr></td>
              </tr></tbody></table></td>
      </tr><tr><td class="diff-line addition" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>89</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc'>+        Requisition<wbr> requisition = new R<wbr>equisition();</wbr></wbr></td>
              </tr></tbody></table></td>
      </tr><tr><td class="diff-line addition" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>90</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc'>+</td>
              </tr></tbody></table></td>
      </tr><tr><td class="diff-line addition" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>91</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc'>+</td>
              </tr></tbody></table></td>
      </tr><tr><td class="line-comment" style="font: 14px/1.4285714 Arial, sans-serif; padding: 10px; background: #f5f5f5; border-top: 1px solid #ccc; border-bottom: 1px solid #ccc">
            <div style="border: 1px solid #ccc; background: #fff; padding: 20px; border-radius: 5px">
              <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="comment" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0">
  <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="comment-avatar" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; width: 32px; vertical-align: top">
          
            
              <img width="32" height="32" alt="rsansores" src="https://secure.gravatar.com/avatar/85b08342365897cc88f655fc08c45a10?d=https%3A%2F%2Fd3oaxc4q5k2d6q.cloudfront.net%2Fm%2F15329b3308f4%2Fimg%2Fdefault_avatar%2F32%2Fuser_blue.png&amp;s=32" style="border-radius: 3px"></td>
        <td class="comment-main" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0 0 0 10px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="comment-user" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; line-height: 1">
                    
                      
                        <strong>Ricardo Sansores</strong> commented on pull request #201:
                      
                    
                  </td>
                </tr><tr><td class="title" style="font: 14px/1.4285714 Arial, sans-serif; padding: 5px 0 0; font-weight: bold; line-height: 1.2">
                      <a href="https://bitbucket.org/globalgassf/globalnet/pull-request/201/requisicion-y-detalle-de-requisicion#comment-6707551" style="color: #3572b0; text-decoration: none">Requisicion y Detalle de requisicion</a>
                    </td>
                  </tr><tr><td class="comment-content markup-content" style="font: 14px/1.4285714 Arial, sans-serif; padding: 10px 0 15px">
                  
                    <p style="margin-bottom: 0; margin: 10px 0 0 0; padding: 0; margin-top: 0">Formatear el código, he encontrado muchos lugares en que hay espacios, saltos de línea y cosas de sobra. El IDE tiene shortcuts para formatear todo y dejarlo perfectamente.</p>
                  
                </td>
              </tr></tbody></table></td>
      </tr><tr><td class="actions secondary" colspan="2" style="font: 14px/1.4285714 Arial, sans-serif; padding: 20px 0 0; color: #707070; border-top: 1px solid #cccccc; line-height: 1">
            
            
              
                <a href="https://bitbucket.org/globalgassf/globalnet/pull-request/201/requisicion-y-detalle-de-requisicion#comment-6707551" style="color: #3572b0; text-decoration: none">View this pull request</a> or add a comment by replying to this email.
              
            
          </td>
        </tr></tbody></table></td>

                    
                  </tr></tbody></table></div>
          </td>
        </tr><tr><td class="diff-line addition" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>92</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc'>+        if ( requis<wbr>itionDetail.getStrin<wbr>g("requisition.id") <wbr>== null || requisiti<wbr>onDetail.getString("<wbr>requisition.id").isE<wbr>mpty()) {</wbr></wbr></wbr></wbr></wbr></wbr></td>
              </tr></tbody></table></td>
      </tr><tr><td class="diff-line addition" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>93</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc'>+</td>
              </tr></tbody></table></td>
      </tr><tr><td class="diff-line addition last" style="font: 14px/1.4285714 Arial, sans-serif; padding: 0; font-size: 11px">
          <table style="width: 100%; border-collapse: collapse"><tbody><tr><td class="diff-linenum from" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-bottom-left-radius: 5px'></td>
                <td class="diff-linenum to" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; background: #f5f5f5; text-align: center; width: 30px; border-right: 1px solid #ccc'>94</td>
                <td class="diff-line-content" style='font: 11px Consolas, Menlo, "Liberation Mono", Courier, monospace; padding: 0; white-space: pre-wrap; background: #cfc; border-bottom-right-radius: 5px'>+            requisi<wbr>tion = createRequisi<wbr>tion(requisition, re<wbr>quisitionDetail.getS<wbr>tring("requisition.d<wbr>epartment.id"), requ<wbr>isitionDetail.getStr<wbr>ing("requisition.emp<wbr>loyee.id"));</wbr></wbr></wbr></wbr></wbr></wbr></wbr></wbr></td>
              </tr></tbody></table></td>
      </tr></tbody></table></div>
            </td>
          </tr></tbody></table></div>
  </td>
</tr>
                  
                
                <tr>
                  

<td style="padding: 20px 0; color: #707070;">
  <table style="width: 100%; border-collapse: collapse;">
    <tbody>
      <tr>
        <td style="padding: 0">
          
            
            
              
              
                
                  <a style="color: #3572b0; text-decoration: none;" href="https://bitbucket.org/globalgassf/globalnet/pull-request/201/unwatch/miguelssf/227040c70998db5fb99dd488deb32b6be69c7311/">Unwatch this pull request</a> to stop receiving email updates.
                
              
            
          
          
        </td>
        <td style="padding: 0">
          
            

  <img width="1" height="1" src="https://bitbucket.org/account/notifications/mark-read/151976913/16da1ef99393fc695bad5056d9523c06bc50e0df/">


          
        </td>
        <td style="text-align: right; width: 100px; padding: 0">
          <a href="https://bitbucket.org" style="color: #3572b0; text-decoration: none;">
            <img width="100" height="18" src="https://d3oaxc4q5k2d6q.cloudfront.net/m/15329b3308f4/img/email/logo.gif" alt="Bitbucket" />
          </a>
        </td>
      </tr>
    </tbody>
  </table>
</td>

                </tr>
              </tbody>
            </table>
          </td>
        </tr>
      </tbody>
    </table>
  </body>
</html>

--===============6761295920308143537==--
