fix: import file with null value (#332)

Author: adrienaury

CHANGELOG.md

@@ -20,6 +20,7 @@ Types of changes
 - `Added` commands `set-parent-select` and `set-child-select` to `lino id`
 - `Added` `import: no` option for columns in `tables.yaml` configuration
 - `Added` `export: presence` option for columns in `tables.yaml` configuration
+- `Fixed` issue with `import: file` when data is null, will not fail anymore
 
 ## [3.0.2]
 

pkg/push/model_table.go

@@ -176,17 +176,14 @@ func (t table) Import(row map[string]interface{}) (ImportedRow, *Error) {
 
 			format, _ := parseFormatWithType(col.Import())
 
-			if format == "file" {
+			if value, exists := result.GetValue(key); format == "file" && exists && value.GetFormat() == jsonline.String && value.Raw() != nil {
 				bytes, err := t.filecache.Load(result.GetString(key))
 				if err != nil {
 					return ImportedRow{}, &Error{Description: err.Error()}
 				}
 				result.SetValue(key, jsonline.NewValueAuto(bytes))
-			}
-
-			// autotruncate
-			value, exists := result.GetValue(key)
-			if exists && col.Truncate() && col.Length() > 0 && value.GetFormat() == jsonline.String && result.GetOrNil(key) != nil {
+			} else if exists && col.Truncate() && col.Length() > 0 && value.GetFormat() == jsonline.String && result.GetOrNil(key) != nil {
+				// autotruncate
 				if col.LengthInBytes() {
 					result.Set(key, truncateUTF8String(result.GetString(key), int(col.Length())))
 				} else {

tests/suites/push/import.yml

@@ -125,6 +125,11 @@ testcases:
           - result.code ShouldEqual 0
           - result.systemout ShouldBeEmpty
           - result.systemerr ShouldBeEmpty
+      - script: lino pull --table staff source | jq -c '.picture=null' | lino push update --table staff dest
+        assertions:
+          - result.code ShouldEqual 0
+          - result.systemout ShouldBeEmpty
+          - result.systemerr ShouldBeEmpty
 
   - name: import no
     steps: