Page context plugin

[jammon]

Ripped parsing of page context out of Page and put it into a plugin. With tests and documentation.
There are different styles of metadata, the plugin tries to guess what is appropriate.
Multimarkdown style is implemented but not used. In the future the style should be up for configuration.

[krallin]

Hi Johannes,

Just a quick note that i'll try to take a look at this over the weekend — sorry about the delayed response here.

Cheers,

[jammon]

I appreciate your work for this project very much, and have learned to be patient in open source development, since I'm well aware, that there are more pressing issues in everybodies life than such a project.
So take your time :)

[krallin]

I think we should refactor this method. data is passed, but we never use it. extra is never passed. How about we remove those altogether?

Cheers,

[jammon]

Sure, I had left it in because somebody else might rely on them being there. But that seems overly cautious.

[krallin]

Hey @jammon ,

I added some inline comments. Most of them are rather minor. The two major things are:

• I'd rather you use preBuildPage in your tests
• We need to make sure this plugin runs first

Cheers,

[jammon]

Thank you for in-depth comments, I agree with all of them.
The plugin now has ORDER = 0. Will that be low enough?

[krallin]

Hey @jammon

Just a quick note that this week / the next one have / are going to be very busy on my end. I'll try to review this but I can't promise anything just yet.

Cheers,

[jammon]

@krallin No problem, currently I have more than enough work to do as well.