Admin Panel

[Dillon]

Is there anyway you could make it so that the admin panel doesnt have all the side bars and top nav/banner and make it like its own separate look like most admin panels because for one it is very crowded looking having all the side bars and what not on the admin panel and two it just plain looks better and would make it seem more professional i guess you could call it. Just a thought.

[kurt]

Have to give that one some thought. The admin panel is integrated with the rest of the script's template system partly so that it can take advantage of the security, without having to maintain it in multiple spots. Let me give it some thought though, see what we can come up with.

[Dillon]

alright

[kurt]

It might be doable by making a different template made up the way you want it, and then modifying the root index.php where it says

Code: [Select]


include ('templates/'.$template.'/template.php');



to something along the lines of

Code: [Select]


if ( $_GET['action'] = 'admin' ) {
include ('templates/'.$template.'/admintemplate.php'); } else {

include ('templates/'.$template.'/template.php');
};


Then all you should need to do is mod the template.php file to suit and save it as admintemplate.php, I think.

[Dillon]

Ah yea that would make sense, i dont have time to test it till next week but if i get it working maybe i can post the admin template for you so u dont have to do the work?

[kurt]

That would be most acceptable to us. Thanks!

[Dillon]

Ok i found this and tried it and what this code does is make it so that it is changed for every single page and not just the admin panel page if your account is an admin. Im going to mess with it some more and try to make it only do it when viewing the admin panel

[Frank]

Try this - works for me.  

index.php

replace the include*.* with

Code: [Select]

if ( $_GET['action'] == 'admin' ) {/*Do Nothing*/} else {
include ('templates/'.$template.'/template.php');
};

pages/admin/index.php

Before the ?> at the bottom add

Code: [Select]

include ('templates/'.$template.'/admin-template.php');
Copy template.php to admin-template.php and chop out what you don't want.

[Dillon]

Wow amazing mort thanks for the great help.

[Frank]

You're welcome and it must have been late at night - - - -

Before the <? add

should have been:

Before the ?> at the bottom add

[kurt]

Thanks!

[lopa]

This script can be improved by changing it to:

Code: [Select]

if (!isset($_GET['action']))
    $_GET['action'] = null;

if ( $_GET['action'] == 'admin' ) {/*Do Nothing*/} else {
include ('templates/'.$template.'/template.php');
};
This prevents the code from trying to read $_GET['action'] when the element may not be defined.

[Adam LaCombe]

Don't hate on me for this lol but you could simplify that.


if(isset($_GET['action']) && $_GET['action'] == 'admin' ){
    include ('templates/'.$template.'/template.php');
}


Just less code to worry about that way, but either way works of course.

[lopa]

Don't hate me for the alternatives.  

The fix you posted only addresses a single element.

I was going to simplify it with something like you suggest  but I had something coming that would more or less keep things consistent with a lot less work for quite a lot of other errors of the same nature that exist in both the front and back end of your script.

And here are just some of them.

Your script is using array elements without checking whether they exists. You should make sure that your code does not try to read things like $_GET['action'] etc, when an array element may not be defined.

And this is a couple of examples of one way of overcoming it.

Code: [Select]

$action = isset($_GET['action']) ? $_GET['action'] : null;

switch ($action) { ... }


$cmd = isset($_GET['cmd']) ? $_GET['cmd'] : null;

switch ($cmd) { ... }


Or, you can fix this in a way that doesn't require altering your switch logic, by giving $_GET['action'] etc, a defined (but "empty") value if action isn't given in a query string:


if (!isset($_GET['action']))
    $_GET['action'] = null;

switch ($_GET['action']) { ... }


if (!isset($_GET['cmd']))
    $_GET['cmd'] = null;

switch ($_GET['cmd']) { ... }


if (!isset($_GET['type']))
    $_GET['type'] = null;

switch ($_GET['type']) { ... }


if (!isset($_GET['page']))
    $_GET['page'] = null;


if (!isset($_GET['name']))
    $_GET['name'] = null;

if (!isset($_GET['case']))
    $_GET['case'] = null;
Needless to say, I've been looking at your script with "full error reporting"; and most of the errors are generated because of insufficient arguments being applied as to whether something or anything exists or not.

Take your pick, but there are a lot of undefined errors, simply because they lack an argument while others are simply not defined.

[Adam LaCombe]

Yeah I know lol its awful.

I am working on V2.1 right now and I been fixing tons of errors and adding error reportings.
Such as if id isn't defined display error and don't continue rest of code'
If id doesn't exist in db display error
etc.

I think the new version will work a lot better then the previous ones.
I have fixed a lot of things so far.
I am about to work on a language php class right now before I finish fixing all the files.

There also is a new template system that will work a lot better then the current or the one before the current.

I'll hopefully have the new release candidate released in three weeks mayyyybe. I am going on vacation in 2 weeks though so no promises.