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

Separate application logic from layout #52

Open
soren121 opened this issue Dec 16, 2014 · 4 comments
Open

Separate application logic from layout #52

soren121 opened this issue Dec 16, 2014 · 4 comments

Comments

@soren121
Copy link
Owner

From [email protected] on August 02, 2012 11:07:28

Right now the dashboard is embedded into the PHP files with application logic intermingled, which is quite frowned upon as the application logic should be separated from the layout.

Either we should switch over to a templating system similar to Smarty or just make separate PHP files which handle displaying of the page with no logic in it.

Original issue: http://code.google.com/p/lightblog/issues/detail?id=52

@soren121
Copy link
Owner Author

From [email protected] on August 13, 2012 13:06:04

(No comment was entered for this change.)

Labels: Maintainability Component-Core

@soren121
Copy link
Owner Author

From [email protected] on August 18, 2012 15:54:05

Just a simple suggestion; keep lightblog light! Use savant3: http://phpsavant.com/

@soren121
Copy link
Owner Author

From [email protected] on August 18, 2012 16:22:53

Apparently Savant was last updated in 2009, and a project called Savvy was started as a replacement: https://github.com/saltybeagle/Savvy The last commits were a year ago.

@soren121
Copy link
Owner Author

From doodle62 on August 18, 2012 17:02:51

I neglected to discuss this in the issue comments, but we won't be using a templating system, especially not for the admin panel. We're doing our best to separate logic from layout, and most of the bigger infractions (like the dashboard that Ian mentioned) have been dealt with already. It really doesn't make sense to use a templating system in the admin panel because it isn't designed to be themeable, and it would severely limit our ability to implement certain features. We will (in the future) offer the ability to use different stylesheets to change its look that way. At the most, we'll split it with a Model-View-Controller (MVC) kind of thing.

@soren121 soren121 self-assigned this Dec 16, 2014
@soren121 soren121 added this to the 1.0 milestone Dec 16, 2014
@soren121 soren121 modified the milestone: 1.5 Dec 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant