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

Agree on strategy for helper functions (Trac #4327) #4327

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

Agree on strategy for helper functions (Trac #4327) #4327

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

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/4327 on 42062150-10-01 by brettp, assigned to unknown.

Elgg version: 1.8

See discussion at #4311

@elgg-gitbot
Copy link
Author

ewinslow wrote on 42543646-12-19

Comment/explanation?

@elgg-gitbot
Copy link
Author

cash wrote on 42543797-07-20

The ticket does not have enough information to be worth keeping around. Plus, I feel like we have a good handle on this: we want to move helper functions into classes (static in a few cases, class methods in most others) plus we we want to wait to do a lot of this until we're at PHP 5.3 and have decided how to organize our classes instead of throwing them all into a single classes directory.

@elgg-gitbot
Copy link
Author

ewinslow wrote on 42544293-04-15

SGTM. Just don't like tickets getting closed without comment.

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