So this is set in the ctor hence why readonly is ok?

Why the explicit set to null? The compiler does that already.