/ - Diff - Application: gvSIG desktop - gvSIG

         public class H2SpatialSelectBuilderBase extends SelectBuilderBase {
             @Override
             protected boolean isValid(StringBuilder message) {
                 if( message == null ) {
                     message = new StringBuilder();
+                }
                 if( this.has_offset() && !this.has_order_by() ) {
                     // Algunos gestores de BBDD requieren que se especifique un
                     // orden para poder usar OFFSET. Como eso parece buena idea para
                     // asegurar que siempre tengamos los mismo resultados, lo exijimos
                     // siempre.
                     message.append("Can't use OFFSET without an ORDER BY.");
                     return false;
+                }
                 return true;
+            }
             @Override
             public String toString(Formatter formatter) {
                 StringBuilder builder = new StringBuilder();
                 if( !isValid(builder) ) {

                 // Probablemente deberiamos tener alguna otra forma de detectar que
                 // estamos paginanado ya que asi no distinguimo si solo queremos
                 // obtener los primeros elementos sin importarnos su orden.
                 for(String attrName : primaryKeys ) {
                     // Se precisa indicar un orden para usar OFFSET.
                     select.order_by().column(sqlbuilder.as_identifier(attrName)).ascending();
                 if( primaryKeys.isEmpty() ) {
                   // Muy probablemente si no tiene pk sea una vista, asi que
                   // pasaremos de ordenar y esperemos que la vista este ya ordenada.
                   select.disable_check_order_and_offset();
                 } else {
                   for(String attrName : primaryKeys ) {
                       // Se precisa indicar un orden para usar OFFSET.
                       select.order_by().column(sqlbuilder.as_identifier(attrName)).ascending();
+                  }
+                }
+            }
             if( limit > 0 ) {
                 select.limit(limit);

             public boolean has_limit();
             public boolean has_offset();
             public void disable_check_order_and_offset();
             /**
              * Constructs the SQL statement. If the values associated with the SQL

             protected List<OrderByBuilder> order_by;
             protected boolean distinct;
             protected List<Variable> groupColumn;
             protected boolean check_order_and_offset = true;
             public SelectBuilderBase() {
                 this.columns = new ArrayList<>();
-...
                 return !this.groupColumn.isEmpty();
+            }
             @Override
             public void disable_check_order_and_offset() {
               this.check_order_and_offset = false;
+            }
             protected boolean isValid(StringBuilder message) {
                 if (message == null) {
                     message = new StringBuilder();
+                }
                 if (this.has_offset() && !this.has_order_by()) {
                     // Algunos gestores de BBDD requieren que se especifique un
                     // orden para poder usar OFFSET. Como eso parece buena idea para
                     // asegurar que siempre tengamos los mismo resultados, lo exijimos
                     // siempre.
                     message.append("Can't use OFFSET without an ORDER BY.");
                     return false;
                 if( this.check_order_and_offset ) {
                   if (this.has_offset() && !this.has_order_by()) {
                       // Algunos gestores de BBDD requieren que se especifique un
                       // orden para poder usar OFFSET. Como eso parece buena idea para
                       // asegurar que siempre tengamos los mismo resultados, lo exijimos
                       // siempre.
                       message.append("Can't use OFFSET without an ORDER BY.");
                       return false;
+                  }
+                }
                 return true;
+            }

Application: gvSIG desktop

Revision 44687