Problem with complex fix using paste + replace_all

[TobiasNx]

hbz/lobid-resources#1584

I have a complex fix that uses paste and replace_all besides others.
while the paste alone does not create the wrong transformation

Adding an replace_all function transforming the results of the pase seem to create an wrong output:

    "altLabel" : [ "Ev.-Luth. Landeskirche Sachsens ( )", "Evang.-Luth. Landeskirche Sachsens ( )", "EVLKS ( )", "Ev.-Luth. Landeskirche Sachsens ( )", "Evang.-Luth. Landeskirche Sachsens ( )", "EVLKS ( )" ],
    "$i" : {
      "altLabel" : [ {
        "1" : "Ev.-Luth. Landeskirche Sachsens",
        "2" : "Evang.-Luth. Landeskirche Sachsens",
        "3" : "EVLKS",
        "4" : "Ev.-Luth. Landeskirche Sachsens",
        "5" : "Evang.-Luth. Landeskirche Sachsens",
        "6" : "EVLKS"
      } ]
      ```

I use the combination with paste and replace_all at another place in the fix but there do not have the problem:
https://github.com/hbz/lobid-resources/pull/1584/commits/733286d2e1f6841f795650c57c18c166c453ce06

[blackwinter]

replace_all() seems to have issues with programmatically added values in arrays (not limited to paste()).

ETA: Also not limited to replace_all(), presumably all "string transformers" are affected.

[blackwinter]

Here are a bunch of test cases: e7ff7fd

@fsteeg: Can you look into fixing the path logic?

@TobiasNx: Do you want to try and supply an integration test for your use case?

[TobiasNx]

Is this also related to this problem: hbz/lobid-resources#1585?

Maybe it is related to this: #250 If we change stuff with replace_all perhaps we should take in mind the other problems

[blackwinter]

Is this also related to this problem: hbz/lobid-resources#1585?

Yes.

Maybe it is related to this: #250

Doesn't look like it, but can't say for sure.

[fsteeg]

@fsteeg: Can you look into fixing the path logic?

I'll take a look, thanks for the unit tests!

[fsteeg]

Pushed some tweaks in 87e30a7 and 416ba75, which seem to fix the use cases from the unit tests.

[TobiasNx]

I added my case here as integration test: #280

[TobiasNx]

Tested my integration test it works in the new branch.