Arne Babenhauserheide
Arne Babenhauserheide
@toad is there any chance that you will get to that? If not, could you give it a regular review (given that you know this part of the code best)?
@toad ping
Can I merge this? @toad
@toad can you have a look at this?
First step: rebased and whitespace and imports cleaned up.
Second step: more cleanup and some force pushing, because it should now be much easier to check.
I am not sure about this one. We used to have plugins that Freesites could embed as iframes. Currently we do not have them, but they are a valid usecase...
I think currently not, but that is a usecase.
> I am skeptical about this. Why should `message` ever be `null`? That code is only used within Fred, right? There’s no user input involved? Because if there isn’t it...
> I traced back every call of that method and I couldn’t see how anything given in to this method can ever be `null`. Damn :-) I also found no...