search

ome / omero-mapr

An OMERO.web app allowing to browse the data through attributes linked to the image
https://pypi.org/project/omero-mapr/
GNU Affero General Public License v3.0
5 stars 12 forks source link

syntax error in mapr_paths_to_object ? #37

Open mtbc opened 6 years ago

mtbc commented 6 years ago

Is it okay that the HQL line at https://github.com/ome/omero-mapr/blob/0.2.1/omero_mapr/show.py#L184 doesn't end with a comma?

mtbc commented 6 years ago

Though, ha, also if https://github.com/ome/omero-mapr/blob/0.2.1/omero_mapr/show.py#L233 doesn't happen because there's no where clause then I suspect that the trailing "and"s from just above would cause a syntax error with the following "group by". Maybe that's less of an issue though.

joshmoore commented 6 years ago

Yeah, the first issue is quite remarkable.