Make recipe keywords an array

[christianlupus]

In a JSON+LD object, the keywords are an array of entries. We are currently mapping that into a comma-separated list of keywords stored in the *.json files. On the frontend, the array is reconstructed and used in a v-for Vue attribute.

Example locations:

• cookbook/lib/Service/RecipeService.php

  Lines 292 to 294 in f299ecc

  	if (isset($json['keywords']) && is_array($json['keywords'])) {
  	$json['keywords'] = implode(',', $json['keywords']);
  	}

• cookbook/src/components/RecipeEdit.vue

  Lines 518 to 523 in f299ecc

  	$this.allKeywords = []
  	for (let i = 0; i < json.length; i++) {
  	if (json[i].name !== "*") {
  	$this.allKeywords.push(json[i].name)
  	}
  	}

I am considering this a bad idea to do such mapping back and forth without any obvious benefit. I do not see such a benefit at the moment. However, I might be misled and there is in fact a reason for this. I am asking for you opinions on this. Any comments, @seyfeb?

[seyfeb]

I don't know of and don't see a reason. I implemented this in the frontend like this only because that was the data the API returned!

[christianlupus]

All right, no offense. I just wanted to make sure, i am not dumb.

I will create a wishlist for that in the issues then.

[seyfeb]

None taken ;)