Skip to content
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

hierarchy of object-based modules framework #4

Closed
wants to merge 9 commits into from
Closed

hierarchy of object-based modules framework #4

wants to merge 9 commits into from

Conversation

ildemartinez
Copy link
Contributor

New custom emoncms module class. Each module should inherit from custom module.
Added input and feed class modules
Admin view shows installed modules
TODO: build menu from module object information (we could have not visible modules)(not necesary in settings to set and know the modules) --->>>> DONE
The menu is build from information from each module object so emocms dont have to know more information (modules abstraction)
The module load/registration logic is moved to a class to encapsulate behaviour
The registration of module is made only when is needed due the singleton pattern
Updated to Twitter Bootstrap 2.1.0
:-)

New custom emoncms module class. Each module should inherit from custom module.
Added input and feed class modules
User view shows installed modules
…class

The module dont need to know the menu variable where get connected
Menu names in classes allows get translation in his own localization file
Faster using singleton pattern for registered modules
Delete emoncms modules constructors (not yet needed)
… behaviour

The registratio of module is made only when is needed due the singleton pattern
@TrystanLea TrystanLea closed this Dec 12, 2012
TrystanLea pushed a commit that referenced this pull request Nov 9, 2015
chaveiro added a commit that referenced this pull request Jan 16, 2016
vkylamba pushed a commit to vkylamba/emoncms that referenced this pull request Mar 17, 2017
vkylamba pushed a commit to vkylamba/emoncms that referenced this pull request Mar 18, 2017
@spike3312 spike3312 mentioned this pull request Dec 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants