Don't append scopeVariable name, set base path on new values (#278)

Tweak test: add blank inserted by `paste` as default `join_char`

Author: fsteeg

metafix/src/main/java/org/metafacture/metafix/FixBind.java

@@ -36,7 +36,7 @@ public void execute(final Metafix metafix, final Record record, final List<Strin
 
                     // with var -> keep full record in scope, add the var:
                     if (scopeVariable != null) {
-                        record.put(scopeVariable, value);
+                        record.put(scopeVariable, value, false);
                         recordTransformer.transform(record);
                         record.remove(scopeVariable);
                     }

metafix/src/main/java/org/metafacture/metafix/FixPath.java

@@ -276,7 +276,7 @@ else if (isReference(field)) {
         }
         else {
             if (!hash.containsField(field)) {
-                hash.put(field, Value.newHash());
+                hash.put(field, Value.newHash().withPathSet(newValue.getPath()));
             }
             insertInto(hash.get(field), mode, newValue, field, tail(path));
         }

metafix/src/test/java/org/metafacture/metafix/MetafixMethodTest.java

@@ -2771,7 +2771,6 @@ public void addFieldWithoutReplaceAllArray() {
     }
 
     @Test
-    @MetafixToDo("See https://github.com/metafacture/metafacture-fix/issues/278")
     public void addFieldWithReplaceAllArray() {
         addFieldWithReplaceAllArray(true);
     }
@@ -2885,7 +2884,6 @@ public void copyFieldWithoutReplaceAllArray() {
     }
 
     @Test
-    @MetafixToDo("See https://github.com/metafacture/metafacture-fix/issues/278")
     public void copyFieldWithReplaceAllArray() {
         copyFieldWithReplaceAllArray(true);
     }
@@ -2943,7 +2941,6 @@ public void pasteWithoutReplaceAll() {
     }
 
     @Test
-    @MetafixToDo("See https://github.com/metafacture/metafacture-fix/issues/278")
     public void pasteWithReplaceAll() {
         pasteWithReplaceAll(true);
     }
@@ -2958,8 +2955,8 @@ private void pasteWithReplaceAll(final boolean replaceAll) {
                 "  set_array('$i.altLabel[]')",
                 "  paste('$i.altLabel[].$append', '$i.label', '~!')",
                 "end",
-                replaceAll ? "replace_all('contribution[].*.agent.altLabel[].*', '!', '')" : "",
-                replaceAll ? "replace_all('subject[].*.altLabel[].*', '!', '')" : ""
+                replaceAll ? "replace_all('contribution[].*.agent.altLabel[].*', ' !', '')" : "",
+                replaceAll ? "replace_all('subject[].*.altLabel[].*', ' !', '')" : ""
             ),
             i -> {
                 i.startRecord("1");