Marc Bernard

dashboard says 19 findings but only 13 listed above. mmm 

kind of a known issues. `zcl_abapgit_html_action_utils` is all custom to process form/post data and has some flaws. this becomes evident if the post data becomes larger and/or has characters that...

the concern was noted in the code `* do not use cl_http_utility as it does strange things with the encoding` but I don't think this was actually proven to be...

https://github.com/abapGit/abapGit/pull/4615 might be related since it's also dealing imho with characters (like `space`) that are not escaped properly.

If this is still reproducible, please follow the instructions here to pinpoint where it goes wrong in the JavaScript: https://docs.abapgit.org/development/debugging.html

https://github.com/abapGit/abapGit/pull/5508 has fixed a few things related to escaping special characters, for example in links. Is this still an issue?

Yes, please check. "JS init" errors are rare but we still want to fix them :-)

AG doesn't keep the object status by branch. Therefore, when switching branches, we can't reliably determine whether the local or the remote side was changed. Removing `not lstate is initial`...

I would agree if we can define how we visualize the diffs. Which side is "green" and which is "red"?

The code looks correct to me. If the local state is `A` or `M`, then the object needs to be added. If it's `D` then it needs to be removed....