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

Remove require_once() for known core handlers from all scripts. #125

Open
CaMer0n opened this issue Feb 27, 2013 · 1 comment
Open

Remove require_once() for known core handlers from all scripts. #125

CaMer0n opened this issue Feb 27, 2013 · 1 comment
Labels
type: enhancement An improvement or new feature request
Milestone

Comments

@CaMer0n
Copy link
Member

CaMer0n commented Feb 27, 2013

eg. one might replace:

 require_once(e_HANDLER."message_handler.php");

with:

$mes = e107::getMessage();

(And test to make sure nothing breaks)

If we can get this done, @SecretR will be happy. 👍

CaMer0n added a commit that referenced this issue Feb 27, 2013
@ghost ghost assigned Moc Feb 27, 2013
@Moc
Copy link
Member

Moc commented Feb 27, 2013

I've been adding them lately.. Oops.
Will work on removing them :)

Moc added a commit that referenced this issue Feb 27, 2013
Moc added a commit that referenced this issue Feb 28, 2013
Moc added a commit that referenced this issue Mar 3, 2013
Moc added a commit that referenced this issue Mar 4, 2013
Moc added a commit that referenced this issue Mar 5, 2013
Moc added a commit that referenced this issue Mar 10, 2013
Moc added a commit that referenced this issue Mar 10, 2013
@CaMer0n CaMer0n modified the milestones: v2.0, Future Sep 2, 2015
MikeyGMT added a commit to MikeyGMT/e107 that referenced this issue Jan 21, 2017
Replace require_once form_handler for userclass2 e107inc#125
MikeyGMT added a commit to MikeyGMT/e107 that referenced this issue Jan 21, 2017
Replace require_once userclass_class e107inc#125
MikeyGMT added a commit to MikeyGMT/e107 that referenced this issue Feb 4, 2017
Replace require_once for userclass e107inc#125 - tested ok. added FIXME note
where it breaks
@Moc Moc removed their assignment Jan 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement An improvement or new feature request
Projects
None yet
Development

No branches or pull requests

2 participants