Stéphane Épardaud

Results 664 comments of Stéphane Épardaud

Yeah, for strings, we might be able to use an empty string to disinguish: - `` →`@RestQuery String foo` will be `null` because it not set in the request -...

> 1. Do we leave the array parameters (single = false) as is, or do we make them null as well? Leaving them as is would make `List` params be...

The TCK complains about cookie params only, perhaps there's a bug in your PR, only for cookies? As for `RuntimeParamConverterTest.sendEmptyParameter` I think it's a pretty convoluted test, for example it...

> What do we do with `foo=&foo=1`? If I understood you correctly, you would expect it to be become `[1]` and not `[null, 1]` Yeah, this is the only thing...

> Do I make a PR to TCK as well? yeah, it's not right that a query param change would break cookie tests. Thta's just bad testing.

I'm not entirely sure what this is about then. Why would an entity delete operation be cancelled, because persist and delete occur in the same transaction? Sounds like an ORM...