changed PageContent.php to accept nested includes - #4192

hi @ssddanbrown, could you please review this pull request?

Thanks in Advance.

Thanks @jasonF1000 for offering this PR. A few of things:

• The code provided seems to have whitespace issues, mainly in that it seems to be using non-breaking spaces instead of normal spaces within the indentation, which breaks this as code.
• This loop only needs to be around parsePageIncludes, not blankPageIncludes too.
• Ideally I'd want to have testing added to cover this scenario (With some depth checking to ensure this sticks to three levels max).

There are existing tests to cover page includes in this file:
https://github.com/BookStackApp/BookStack/blob/development/tests/Entity/PageContentTest.php
Would you be comfortable in adding the requested testing?

Note-to-self

If merged, this will need to be documented as a breaking change.

Hi @ssddanbrown i followed your recommendation and fixed the space issue and change the loop that it is only around the parsePageIncludes.

But i feel not so comfortable to add the testings, therefore i have to low experience with PHP. Sorry :(

Thanks @jasonF1000, now merged for next release.
I followed with up with 56f234d to adjust formatting and add testing.

@ssddanbrown thank you for taking over the testing part and for the implementation of the feature :)