Merge pull request #5462 from rldhont/fix-constraint-geom

[Fix] QGIS constraint with geometry

Author: rldhont

lizmap/modules/lizmap/lib/Form/QgisForm.php

@@ -229,7 +229,7 @@ public function __construct($layer, $form, $featureId, $loginFilteredOverride, A
                     if ($wkt && \lizmapWkt::check($wkt)) {
                         $geom = \lizmapWkt::parse($wkt);
                         if ($geom === null) {
-                            \jLog::log('Parsing WKT failed! '.$wkt, 'error');
+                            $this->logMessage('Parsing WKT failed: "'.$wkt.'".', 'lizmapadmin');
                         }
                     }
 
@@ -253,6 +253,19 @@ public function __construct($layer, $form, $featureId, $loginFilteredOverride, A
         $form->getContainer()->privateData = array_merge($form->getContainer()->privateData, $privateData);
     }
 
+    /**
+     * Log a Form message.
+     *
+     * @param mixed  $message The message to log
+     * @param string $cat     The category of the logged message
+     */
+    protected function logMessage($message, $cat = 'default'): void
+    {
+        $project = $this->layer->getProject();
+        $prefix = 'Error in form '.$project->getRepository()->getKey().' / '.$project->getKey().' / '.$this->layer->getName().': ';
+        $this->appContext->logMessage($prefix.$message, $cat);
+    }
+
     /**
      * Get the default value of a QGIS field, if this is
      * a simple raw value.
@@ -751,9 +764,23 @@ public function check($feature = null)
 
         // Evaluate constraint expressions
         if (count($constraintExpressions) > 0) {
+
+            // build geom
+            $geom = null;
+            if ($geometryColumn != '') {
+                // from wkt to geom
+                $wkt = trim($form->getData($values[$geometryColumn]));
+                if ($wkt && \lizmapWkt::check($wkt)) {
+                    $geom = \lizmapWkt::parse($wkt);
+                    if ($geom === null) {
+                        $this->logMessage('Parsing WKT failed: "'.$wkt.'".', 'lizmapadmin');
+                    }
+                }
+            }
+
             $form_feature = array(
                 'type' => 'Feature',
-                'geometry' => null,
+                'geometry' => $geom,
                 'properties' => $values,
             );
             $results = $this->evaluateExpression($constraintExpressions, $form_feature);
@@ -922,14 +949,14 @@ public function saveToDb($feature = null, $modifiedControls = array())
             if ($insertAction) {
                 // For insertion, one field has to be set
                 // FIXME missing context
-                $this->appContext->logMessage('Error in form, SQL cannot be constructed: no fields available for insert !', 'lizmapadmin');
+                $this->logMessage('SQL cannot be constructed: no fields available for insert.', 'lizmapadmin');
                 $this->form->setErrorOn($geometryColumn, \jLocale::get('view~edition.message.error.save').' '.\jLocale::get('view~edition.message.error.save.fields'));
 
                 // do not throw an exception to let the user update the form
                 throw new \Exception($this->appContext->getLocale('view~edition.link.error.sql'));
             }
             // For update, nothing has changed so nothing to do except close form
-            $this->appContext->logMessage('SQL cannot be constructed: no fields available for update !', 'lizmapadmin');
+            $this->logMessage('SQL cannot be constructed: no fields available for update.', 'lizmapadmin');
 
             return true;
         }
@@ -994,7 +1021,7 @@ public function saveToDb($feature = null, $modifiedControls = array())
                     } catch (\Exception $e) {
                         // Need to catch Exception if operation on remote storage fails
                         $form->setErrorOn($ref, $e->getMessage());
-                        $this->appContext->logMessage($e->getMessage(), 'lizmapadmin');
+                        $this->logMessage($e->getMessage(), 'lizmapadmin');
                         $this->appContext->logException($e, 'lizmapadmin');
 
                         return false;
@@ -1042,7 +1069,7 @@ public function saveToDb($feature = null, $modifiedControls = array())
             return $pkVal;
         } catch (\Exception $e) {
             $form->setErrorOn($geometryColumn, $this->appContext->getLocale('view~edition.message.error.save'));
-            $this->appContext->logMessage('An error has been raised when saving form data edition to db : ', 'lizmapadmin');
+            $this->logMessage('An error has been raised when saving form data edition to the database.', 'lizmapadmin');
             $this->appContext->logException($e, 'lizmapadmin');
 
             return false;
@@ -1505,7 +1532,7 @@ protected function fillControlFromUniqueValues($fieldName, $formControl)
         if (array_key_exists('notNull', $formControl->uniqueValuesData)
             && $formControl->uniqueValuesData['notNull']
         ) {
-            $this->appContext->logMessage('notNull '.$formControl->uniqueValuesData['notNull'], 'lizmapadmin');
+            $this->logMessage('notNull '.$formControl->uniqueValuesData['notNull'], 'lizmapadmin');
             $formControl->ctrl->required = true;
         }
         // combobox
@@ -1539,7 +1566,7 @@ private function fillControlFromValueRelationLayer($fieldName, $formControl)
         if (array_key_exists('notNull', $formControl->valueRelationData)
                 and $formControl->valueRelationData['notNull']
         ) {
-            \jLog::log('notNull '.$formControl->valueRelationData['notNull'], 'lizmapadmin');
+            $this->logMessage('notNull '.$formControl->valueRelationData['notNull'], 'lizmapadmin');
             $formControl->ctrl->required = true;
         }
         // combobox

tests/end2end/playwright/axis_orientation.spec.js

@@ -51,6 +51,9 @@ test.describe('Axis Orientation',
             const contentLength = await getMapResponse?.headerValue('Content-Length');
             expect(parseInt(contentLength ? contentLength : '0')).toBeGreaterThan(5552);
 
+            // Wait for transition
+            await page.waitForTimeout(1000);
+
             buffer = await page.screenshot({clip:{x:950/2-380/2, y:600/2-380/2, width:380, height:380}});
             const bundeslanderByteLength = buffer.byteLength;
             await expect(bundeslanderByteLength).toBeGreaterThan(blankByteLength);
@@ -142,6 +145,9 @@ test.describe('Axis Orientation',
             // image size lesser than disorder axis
             expect(parseInt(contentLength ? contentLength : '0')).toBeLessThan(240115);
 
+            // Wait for transition
+            await page.waitForTimeout(1000);
+
             buffer = await page.screenshot({clip:{x:950/2-380/2, y:600/2-380/2, width:380, height:380}});
             const judetByteLength = buffer.byteLength;
             await expect(judetByteLength).toBeGreaterThan(blankByteLength);

tests/units/classes/Form/QgisFormTest.php

@@ -508,7 +508,7 @@ public function testFillControlFromUniqueValue($uniqueValues, $required, $setAtt
         }
         $control->uniqueValuesData = $uniqueValues;
         $control->ctrl->required = $required;
-        $layer = new QgisLayerForTests();
+        $layer = $this->setUpEnv('test', 'test', $dbFieldValues);
         $layer->dbFieldValues = $dbFieldValues;
         $form->setLayer($layer);
         $form->appContext = new ContextForTests();