FunCMS - Beta release

[Merijn]

Good luck with learning PHP, here some tips.

This is your 'beveiliging.php' in the housekeeping:

if($userinfo['rank'] < 5) {
Header("Location: ../ingelogd");
} 

So, what would happen if I disable the location header? It would do let me create a news message if I want to, haha. Put a exit; into the end of a header("Location: "); otherwise, it would be vulnerable =P.

Maybe some other useful tips,
1. Use a template engine such as smarty or raintpl.
2. use require for important files such as configuration files.
3.

$gebruikersnaam = $_SESSION['gebruikersnaam']; 

Maybe you need to check if the session ['gebruikersnaam']; exists, before you create the session? Could lead to E_NOTICE messages. I guess that's why you're using error_reporting(0); =P.

4. You put a query in the global file, so every page you're loading, there will be one query executed? Could be done better.

There is much more I can say, but I guess you have enough now.

Goodluck!

[KevinZuiker]

Good luck with learning PHP, here some tips.

This is your 'beveiliging.php' in the housekeeping:

if($userinfo['rank'] < 5) {
Header("Location: ../ingelogd");
} 

So, what would happen if I disable the location header? It would do let me create a news message if I want to, haha. Put a exit; into the end of a header("Location: "); otherwise, it would be vulnerable =P.

Maybe some other useful tips,
1. Use a template engine such as smarty or raintpl.
2. use require for important files such as configuration files.
3.

$gebruikersnaam = $_SESSION['gebruikersnaam']; 

Maybe you need to check if the session ['gebruikersnaam']; exists, before you create the session? Could lead to E_NOTICE messages. I guess that's why you're using error_reporting(0); =P.

4. You put a query in the global file, so every page you're loading, there will be one query executed? Could be done better.

There is much more I can say, but I guess you have enough now.

Goodluck!

Thanks, I will use this in my next CMS :)

[UnstucK]

It's pretty nice even though it's looking like an edit which is not good at all.

Anyway the Index & Regist page are horribly made and don't even like those but you get my respect for the rest of the pages because they are nice for sure!

Keep it up brother.

[maxwellEU]

Thanks i quite like it cant wait for next release

Sent from my RM-825_eu_euro1_310 using Tapatalk

[madsmartapple]

It's not that bad but I honestly really am not a big fan of it. It's a bit plain.

[Atakan Aydin]

Nice!

[NoBrain]

Design is too clustered, I prefer simplicity over a clustered heavy jQuery design. I will review the PHP later.

Sent from my Windows Phone 8S by HTC using Tapatalk

[Rizzie]

The /me looks a little like HoloCMS, because of that Help tool thing.

[Sir Jacob]

Blegh.. that index.. I'll give you a 7.