Bugfix distance field #145
Conversation
| ) | ||
| with_latlng | ||
| .select("#{distance_column_name} AS #{self.distance_column_name}, #{table_name}.*") | ||
| .order(Arel.sql(distance_column_name).send(options[:reverse] ? 'desc' : 'asc')) |
There was a problem hiding this comment.
Layout/DotPosition: Place the . on the previous line, together with the method call receiver.
Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Metrics/LineLength: Line is too long. [89/80]
| Arel.sql(distance_column_name).send(options[:reverse] ? 'desc' : 'asc') | ||
| ) | ||
| with_latlng | ||
| .select("#{distance_column_name} AS #{self.distance_column_name}, #{table_name}.*") |
There was a problem hiding this comment.
Layout/DotPosition: Place the . on the previous line, together with the method call receiver.
Metrics/LineLength: Line is too long. [93/80]
| with_latlng | ||
| .select("#{distance_column_name} AS #{self.distance_column_name}, #{table_name}.*") | ||
| .where(bound_conditions(bounds)) | ||
| .where(distance_conditions(options)) |
There was a problem hiding this comment.
Layout/DotPosition: Place the . on the previous line, together with the method call receiver.
|
|
||
| with_latlng | ||
| .select("#{distance_column_name} AS #{self.distance_column_name}, #{table_name}.*") | ||
| .where(bound_conditions(bounds)) |
There was a problem hiding this comment.
Layout/DotPosition: Place the . on the previous line, together with the method call receiver.
| distance_column_name = distance_sql(origin, units, formula) | ||
|
|
||
| with_latlng | ||
| .select("#{distance_column_name} AS #{self.distance_column_name}, #{table_name}.*") |
There was a problem hiding this comment.
Layout/DotPosition: Place the . on the previous line, together with the method call receiver.
Metrics/LineLength: Line is too long. [93/80]
bricesanchez
force-pushed
the
bugfix-distance-field
branch
from
3dd5088
to
3f6bb58
Compare
No description provided.