Envelope-to: miguel@ssf.com.mx
Delivery-date: Wed, 25 Nov 2015 22:05:15 -0600
Date: Wed, 25 Nov 2015 22:04:35 -0600
From: Rogelio Delgado <issuessicom@gmail.com>
Reply-To: noreply@108.161.137.201
To: miguel@ssf.com.mx
Message-ID: <565684d35c840_9203fe8a9720b7480664@develop.mail>
References: <merge_request_70@108.161.137.201>
Subject: iCopAPI | Change for calculate locations (#1)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_565684d35ae00_9203fe8a9720b74805e";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
 
 Content preview:  New comment for Merge Request 1 http://108.161.137.201:9000/Rogelio/icopapi/merge_requests/1#note_113
    Rogelio Delgado [...] 
 
 Content analysis details:   (1.2 points, 5.0 required)
 
  pts rule name              description
 ---- ---------------------- --------------------------------------------------
  0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
                             See
                             http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
                              for more information.
                             [URIs: 108.161.137.201]
 -0.7 RCVD_IN_DNSWL_LOW      RBL: Sender listed at http://www.dnswl.org/, low
                             trust
                             [209.85.214.178 listed in list.dnswl.org]
  0.0 FREEMAIL_FROM          Sender email is commonly abused enduser mail provider
                             (issuessicom[at]gmail.com)
 -0.0 SPF_PASS               SPF: sender matches SPF record
 -0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)
                             [209.85.214.178 listed in wl.mailspike.net]
  0.0 WEIRD_PORT             URI: Uses non-standard port number for HTTP
  0.0 NORMAL_HTTP_TO_IP      URI: URI host has a public dotted-decimal IPv4
                             address
  0.0 HTML_MESSAGE           BODY: HTML included in message
 -0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
                             domain
  0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
 -0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
 -0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
  2.0 KAM_BADIPHTTP          Due to the Storm Bot Network, IPs in emails is bad
X-Spam-Flag: NO


----==_mimepart_565684d35ae00_9203fe8a9720b74805e
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

New comment for Merge Request 1

http://108.161.137.201:9000/Rogelio/icopapi/merge_requests/1#note_113


Rogelio Delgado

Tengo duda en esta linea. Seg=C3=BAn leo, el m=C3=A9todo findByPositionWi=
thing regresa una lista de Complaints que est=C3=A9n dentro del Circulo d=
e la zona de inter=C3=A9s y se asigna a la lista complaintListMatch. Esto=
 se realiza en un for, donde se hace esta misma operaci=C3=B3n por cada z=
ona de inter=C3=A9s, pero en cada iteraci=C3=B3n se est=C3=A1 asignando, =
o planchando, la lista de complaintsListMatch por cada zona. Al final sol=
o tendr=C3=A1s los Complaints de la ultima zona. Creo que en lugar de hac=
er esto:

complaintListMatch =3D complaintRepository.findByPositionWithin(interestZ=
one.generateCircle());
	=

Deber=C3=ADas hacer algo como:

complaintListMatch.addAll(complaintRepository.findByPositionWithin(intere=
stZone.generateCircle()));
	=


----==_mimepart_565684d35ae00_9203fe8a9720b74805e
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<html lang=3D'en'>
<head>
<meta content=3D'text/html; charset=3Dutf-8' http-equiv=3D'Content-Type'>=

<title>
GitLab
</title>
</meta>
</head>
<style>
  p.details {
    font-style:italic;
    color:#777
  }
  .footer p {
    font-size:small;
    color:#777
  }
</style>
<body>
<div class=3D'content'>
<p class=3D'details'>
New comment on diff for
<a href=3D"http://108.161.137.201:9000/Rogelio/icopapi/merge_requests/1#n=
ote_113">src/main/java/mx/ssf/icopapi/controllers/ClaimantController.java=
</a>
:
</p>
<div>
<p>Tengo duda en esta linea. Seg=C3=BAn leo, el m=C3=A9todo findByPositio=
nWithing regresa una lista de Complaints que est=C3=A9n dentro del Circul=
o de la zona de inter=C3=A9s y se asigna a la lista complaintListMatch. E=
sto se realiza en un for, donde se hace esta misma operaci=C3=B3n por cad=
a zona de inter=C3=A9s, pero en cada iteraci=C3=B3n se est=C3=A1 asignand=
o, o planchando, la lista de complaintsListMatch por cada zona. Al final =
solo tendr=C3=A1s los Complaints de la ultima zona. Creo que en lugar de =
hacer esto:</p>

<p>complaintListMatch =3D complaintRepository.findByPositionWithin(intere=
stZone.generateCircle());</p>

<p>Deber=C3=ADas hacer algo como:</p>

<p>complaintListMatch.addAll(complaintRepository.findByPositionWithin(int=
erestZone.generateCircle()));</p>

</div>


</div>
<div class=3D'footer' style=3D'margin-top: 10px;'>
<p>
=E2=80=94
<br>
You're receiving this notification because you are a member of the Rogeli=
o Delgado / iCopAPI project team.
<a href=3D"http://108.161.137.201:9000/Rogelio/icopapi/merge_requests/1#n=
ote_113">View it on GitLab</a>
</p>
</div>
</body>
</html>

----==_mimepart_565684d35ae00_9203fe8a9720b74805e--