[api] fix editing of issue tracker entries

[adrianschroeter]

eg. via

# osc api -e /issue_trackers/$tracker
we need to backport in case 2.10 is also affected.

Also all other places in test suite where "put ...., params: $body_data"
is used needs to be checked if we broke the api there as well.
needs to become raw_put again...

[hennevogel]

Maybe you can explain what the problem actually is that you are fixing?

[adrianschroeter]

that it isn't working basically and all changes get ignored when use as described in the api. Just run the adapted test suite without the fix to see that (sending it as www encoded params was never intended in the api).

[hennevogel]

So you do want to get rid of the (partial) json API?

[hennevogel]

How about this?

[vpereira]

to be consistent, create and destroy should return the XML normally generated by the method render_ok.

[hennevogel]

@vpereira please have another look

[codecov]

Codecov Report

Merging #8346 into master will increase coverage by 0.03%.
The diff coverage is 95.45%.

@@            Coverage Diff             @@
##           master    #8346      +/-   ##
==========================================
+ Coverage   90.95%   90.98%   +0.03%     
==========================================
  Files         507      507              
  Lines       20532    20518      -14     
==========================================
- Hits        18674    18669       -5     
+ Misses       1858     1849       -9

[hennevogel]

@adrianschroeter this changes the API now, see the included chunk for api.txt. I think it makes sense, are you okay with that?

[adrianschroeter]

@hennevogel seems you forgot to commit the chunk for api.txt?

If it is just about the drop of .json, I am fine with it (it seems to not usable anyway so far at least for writing).

[hennevogel]

Indeed I have. pushed now