Add additional option to include initialization script for Html.React

[BartAdv]

In my scenario I was using react component in a page that was rendered using partial views fetched via AJAX, so one call for @Html.ReactInitJavaScript() at the beginning was not enough. I wanted to run initialization script just after rendering server-side content and this additional flag seemed to fulfill my requirements.

[Daniel15]

I don't imagine this being a very common use case so I'd prefer having this as a separate helper method (Html.ReactWithInitialize or something like that?) to keep the core Html.React helper clean and free from arbitrary parameters.

[Daniel15]

Also what happens if you render it with the init code here, and then also have a Html.ReactInitJavaScript call on the page? I think the same init code will be rendered twice. You may need to add an _initRendered flag on the component that's set when the init code is rendered so it's only ever rendered twice.

[BartAdv]

Agreed it's highly uncommon scenario, will refactor it.

However I've got some issues with the flag. As I see it, it would have to be a property (but we can't make it with internal setter), because it shouldn't be set by component itself when we call RenderJavaScript (as it would make it side-effecting). And property with public setter would be useless. Maybe Environment should maintain the list of components and keep this additional information there? Or maybe there is no need for such check, after all, if someone is explicitly rendering initialization code after rendering component, he's aware why he's doing so.

[Daniel15]

I wonder if having this as a separate method adds value, it may be better to just inline the Environment.CreateComponent call in both of the methods below.

[BartAdv]

And then it would check if component haven't been rendered already?

[Daniel15]

Thanks for your contribution! I implemented a very slightly modified version (it was mostly the same, just some small changes) along with a unit test. I'll credit you in the release notes :)