Merge pull request #24 from netgen/bugfix-required-field

Make sure validation fails if the field is required, but no media has been selected
netgen · Jan 19, 2018 · 6c9c789 · 6c9c789
2 parents 64efc9a + 75716b2
commit 6c9c789
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/ezpublish_legacy/ngremotemedia/datatypes/ngremotemedia/ngremotemediatype.php b/ezpublish_legacy/ngremotemedia/datatypes/ngremotemedia/ngremotemediatype.php
@@ -34,6 +34,16 @@ function __construct()
      */
     public function validateObjectAttributeHTTPInput( $http, $base, $contentObjectAttribute )
     {
+        $contentObjectAttributeID = $contentObjectAttribute->attribute( 'id' );
+
+        $resourceId = trim( $http->postVariable( $base . '_media_id_' . $contentObjectAttributeID, '' ) );
+
+        if ( (empty($resourceId) || $resourceId === 'removed') && $contentObjectAttribute->validateIsRequired() )
+        {
+            $contentObjectAttribute->setValidationError( ezpI18n::tr( 'extension/ngremotemedia/datatypes', 'It is required to select media.' ) );
+            return eZInputValidator::STATE_INVALID;
+        }
+
         return eZInputValidator::STATE_ACCEPTED;
     }