Merge branch 'issues'

Author: ako

mdl-examples/doctype-tests/17-custom-widget-examples.mdl

@@ -10,7 +10,7 @@
 --   3. COMBOBOX enum       — Enum attribute                (KNOWN BUG: CE1613)
 --   4. COMBOBOX association — DataSource + CaptionAttribute (FIXED: Issue #21)
 --   5. IMAGE basic         — Default pluggable Image widget
---   6. IMAGE with URL      — ImageUrl mode with dimensions
+--   6. IMAGE dimensions    — Custom pixel dimensions + alt text
 --   7. IMAGE with OnClick  — Action binding on click
 --
 -- Known engine bugs (do not affect MDL syntax correctness):
@@ -238,20 +238,22 @@ CREATE PAGE CWTest.P_Image_Basic
   }
 }
 
--- MARK: IMAGE with URL
+-- MARK: IMAGE with dimensions
 
 -- ============================================================================
--- Test 6: IMAGE with URL — ImageUrl mode with custom dimensions
+-- Test 6: IMAGE with custom dimensions
 -- Result: Pending validation
 -- ============================================================================
 
 /**
- * Page with an Image widget configured in imageUrl mode,
- * displaying an external URL with fixed pixel dimensions.
+ * Page with an Image widget with custom pixel dimensions.
+ * Note: ImageUrl mode (imageUrl TextTemplate) is not yet supported —
+ * the template's imageUrl property is null and cannot be created from
+ * scratch without triggering CE0463.
  */
-CREATE PAGE CWTest.P_Image_Url
+CREATE PAGE CWTest.P_Image_Dimensions
 (
-  Title: 'Image URL',
+  Title: 'Image Dimensions',
   Layout: Atlas_Core.Atlas_Default,
   Folder: 'CustomWidgets'
 )
@@ -260,8 +262,6 @@ CREATE PAGE CWTest.P_Image_Url
     ROW row1 {
       COLUMN col1 (DesktopWidth: 12) {
         IMAGE imgBanner (
-          ImageType: imageUrl,
-          ImageUrl: 'https://example.com/banner.png',
           AlternativeText: 'Company banner',
           WidthUnit: pixels,
           Width: 800,

mdl/executor/widget_engine.go

@@ -227,30 +227,12 @@ func setTextTemplateValue(val bson.D, text string) bson.D {
 			if tmpl, ok := elem.Value.(bson.D); ok && tmpl != nil {
 				result = append(result, bson.E{Key: "TextTemplate", Value: updateTemplateText(tmpl, text)})
 			} else {
-				// TextTemplate was null — create a new one
-				result = append(result, bson.E{Key: "TextTemplate", Value: bson.D{
-					{Key: "$ID", Value: mpr.IDToBsonBinary(mpr.GenerateID())},
-					{Key: "$Type", Value: "Forms$ClientTemplate"},
-					{Key: "Fallback", Value: bson.D{
-						{Key: "$ID", Value: mpr.IDToBsonBinary(mpr.GenerateID())},
-						{Key: "$Type", Value: "Texts$Text"},
-						{Key: "Items", Value: bson.A{int32(3)}},
-					}},
-					{Key: "Parameters", Value: bson.A{int32(2)}},
-					{Key: "Template", Value: bson.D{
-						{Key: "$ID", Value: mpr.IDToBsonBinary(mpr.GenerateID())},
-						{Key: "$Type", Value: "Texts$Text"},
-						{Key: "Items", Value: bson.A{
-							int32(3),
-							bson.D{
-								{Key: "$ID", Value: mpr.IDToBsonBinary(mpr.GenerateID())},
-								{Key: "$Type", Value: "Texts$Translation"},
-								{Key: "LanguageCode", Value: "en_US"},
-								{Key: "Text", Value: text},
-							},
-						}},
-					}},
-				}})
+				// TextTemplate was null in the template — skip.
+				// Creating a TextTemplate from null triggers CE0463 because Studio Pro
+				// detects the structural change. The template must be extracted from a
+				// widget that already has this property configured in Studio Pro.
+				log.Printf("warning: opTextTemplate: skipping null TextTemplate (cannot create from scratch without CE0463)")
+				result = append(result, elem)
 			}
 		} else {
 			result = append(result, elem)