New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend the capabilities of the plugins defined by the Plugins configuration property #506

Merged
merged 3 commits into from Aug 18, 2014

Conversation

Projects
None yet
2 participants
@amolenaar
Collaborator

amolenaar commented Aug 17, 2014

Plugins were only able to register responders and symbol types. Now also wiki page factories, test systems and slim tables and comparators can be registered.

Also plugin classes are instantiated before their methods are accessed. This way we can pass on system properties and ComponentFactory, which can help instantiate classes in a standard manner.

The static method access still works, though.

amolenaar added some commits Aug 17, 2014

Renamed TestSystemFactoryRegistrar and WikiPageFactoryRegistrar to -R…
…egistry.

This is in line with other registries in FitNesse.
Plugins are now instantiated before their register functions are called.
This way we can provide a ComponentFactory or Properties to it, for further configuration.

@amolenaar amolenaar added this to the Next release milestone Aug 17, 2014

@amolenaar amolenaar changed the title from Extend the capabilities of the plugins defined by the Plugins configutation property to Extend the capabilities of the plugins defined by the Plugins configuration property Aug 18, 2014

@amolenaar amolenaar merged commit 0dc6af1 into unclebob:master Aug 18, 2014

@int3l

This comment has been minimized.

Show comment
Hide comment
@int3l

int3l Aug 20, 2014

TYPO Should rather be "Slim tables" instead "Sim tables".

TYPO Should rather be "Slim tables" instead "Sim tables".

This comment has been minimized.

Show comment
Hide comment
@amolenaar

amolenaar Aug 21, 2014

Collaborator

Thanks. I corrected it.

Collaborator

amolenaar replied Aug 21, 2014

Thanks. I corrected it.

@amolenaar amolenaar deleted the amolenaar:plugins branch Sep 17, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment