Array of strings handling not working

[TobiasNx]

Separated this issue from #80 to create smaller issues.

Fix is not able to operate with array of strings, It cannot create new ones and cannot refrence existing once.

It should be able to refrence a string/value with e.g.: test.$append, test.* See: http://librecat.org/Catmandu/#wildcards
It would operate on them as items. (This could also be true for handling entire objects as items.)

Broken workflow

It works with a subfield: here

It could be that this issue is too connected to #80 then we should close it.

[fsteeg]

Deployed to test, see your example.

[TobiasNx]

It seems to work with string: +1

But is does not seem to work with array of objects where you add a new object:
here

Should be handled in new ticket.

[fsteeg]

But is does not seem to work with array of objects where you add a new object

Handled this here / in #87 as it's very related. Deployed to test: issue on prod, fixed on test.

[blackwinter]

@TobiasNx: Can you assign me when #87 is ready to be merged?

[TobiasNx]

Somehow the playground examples do not work for me right now. Perhaps due to changes in test.

But I also tried it with these tests here with fix build from 82-append with ./gradlew publishToMavenLocal:
https://gitlab.com/oersi/oersi-etl/-/tree/FunctionalReview/data/experimental/appendArrays

Now none of my examples seems to work. Did I do anything wrong?

[fsteeg]

I don't understand, still looks good to me, as linked in #82 (comment):

issue on prod

{
  "arrayOfObjectimals" : [ {
    "name" : "cat"
  }, {
    "name" : "dog"
  }, {
    "name" : "fox"
  }, "duck" ]
}

fixed on test

{
  "arrayOfObjectimals" : [ {
    "name" : "cat"
  }, {
    "name" : "dog"
  }, {
    "name" : "fox"
  }, {
    "name" : "duck"
  } ]
}

[TobiasNx]

Deployed to test, see your example.

It is this string-specific example, that does not work anymore. It worked when I reviewed it last week.

It outputs:

{
  "arrayOfStringimals" : [ "cat", "dog", "fox", {
    "$append" : "duck"
  } ]
}

Also see these testcases which also encompase copy_field and move_field:
https://github.com/oersi/oersi-etl/blob/96a8260df6262079c1fb88f419d935f4cc8be927/data/experimental/appendArrays

Here as playground example

[fsteeg]

Alright, I think the string cases are now all working: one, two, three.

[TobiasNx]

Looks good now +1