Add render_model_add_block tag

[mkoistinen]

OK, this works nicely to satisfy #3873, but would be even better if ClassyTags would let me declare the nodelist block "argument" as, optional. Anyone know if this is possible?

[mkoistinen]

@ojii, is it possible to have a single, but optional block/nodelist defined in an InclusionTag?

[mkoistinen]

Thats odd. The GitHub integration with Travis says this build failed, but if you click through, it shows all passing tests...

[yakky]

Travis, go home, you're drunk! :)

[yakky]

@mkoistinen can you revert the doc change regarding render_model_add? I have a fix for that
Or just leave it and I'll rebase my PR on top of this after the merge

[ojii]

@ojii, is it possible to have a single, but optional block/nodelist defined in an InclusionTag?

No, not without a heavy dose of dark magic. "endblock detection" happens at parse time, not at render time. Argument processing (including the one that would signal if it should scan for an endblock or not) happens at render time.

[mkoistinen]

@yakky - done
@ojii, ok, thanks, this implementation will stand then.

[coveralls]

Coverage increased (+0.04%) to 90.94% when pulling b7ab3a1 on mkoistinen:mkoistinen-feature/render_model_add_block into 4a75fae on divio:develop.

[coveralls]

Coverage increased (+0.04%) to 90.94% when pulling b7ab3a1 on mkoistinen:mkoistinen-feature/render_model_add_block into 4a75fae on divio:develop.

[coveralls]

Coverage decreased (-0.07%) to 90.834% when pulling b7ab3a1 on mkoistinen:mkoistinen-feature/render_model_add_block into 4a75fae on divio:develop.