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

clean up the boot process (Trac #4317) #4317

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 3 comments
Closed

clean up the boot process (Trac #4317) #4317

elgg-gitbot opened this issue Feb 16, 2013 · 3 comments

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/4317 on 42058036-04-01 by cash, assigned to cash.

Elgg version: 1.8

Here are the issues:

  1. library loading has two for loops when only one is needed
  2. library loading should be order independent (rather than requiring actions.php and database.php to be loaded first)
  3. there is a mix of functions being called directly from start.php and functions registered for the 'boot', 'system' event
  4. there is code in start.php that can be moved into functions
  5. there is confusion between loading configuration for Elgg as an application (think datalists stuff) and as a site (think config table or ElggSite entity).

I was trying to working on caching and the way the code was distributed made it difficult to understand/change.

My attempt at cleaning this up: https://github.com/cash/Elgg/tree/boot

@elgg-gitbot
Copy link
Author

Milestone changed to Elgg 1.8.4 by cash on 42061140-02-07

@elgg-gitbot
Copy link
Author

trac user Cash Costello wrote on 42067385-08-19

Merge pull request #150 from cash/boot

Fixes #4317 cleans up the boot process
Changeset: 54bef9f

@elgg-gitbot
Copy link
Author

trac user Cash Costello wrote on 42088991-01-17

Merge pull request #150 from cash/boot

Fixes #4317 cleans up the boot process
Changeset: 54bef9f

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

No branches or pull requests

1 participant