Habbo4.1 [PHP5, OOP, TPL, MySQLi]

[Dannie]

This cms works perfectly with IIS!

[LeChris]

A CMS isn't so much for design. A CMS has the functions which you need, and should be easy to edit. This CMS achieves that in my eyes, the design is easy to edit, and shouldn't take much time. Honestly, rather than waiting on the dev to finish the other two themes why not make your own or steal one and use it lol.

[Boston21]

Did you have to do anything specific to get it to work with IIS?

[Dannie]

No ..... :)

[Rizzie]

Very nice! Little but nice.

[Jamal7]

Tnxx for this release maybe an good layout for an RP?

[Finley7]

This cms works perfectly with IIS!

Glad to hear, hadn't tested yet.

Sent from my GT-I8260 using Tapatalk

[bugme]

Jamal:

Don't think it's a layout that you'd use for roleplay hotels but anyway good luck!

[Vaulient]

Actually it is better off for a rp.

[Finley7]

Going to update it tomorrow with the latest version. It now has cached avatars.

[Gangnam]

Actually it is better off for a rp.

And how do you work that out?

[Vaulient]

The skin itself , it's more suitable for a rp

[Finley7]

Update 13:

Spoiler:

- Fixed SSO problem

https://mega.co.nz/#!x1JhWAxD!EERvuQ...EeNQxIUvZHKKfI

[Phosfor]

Have you already tried RainTPL for your template system ?

I use it for personal purpose, it's lightweight ( compared to the famous
Smarty and others ) but powerful and easy to use.

[Finley7]

I used it on earlier projects. I wanted to create my own engine now.

[Finley7]

Busy on new theme!

Spoiler:

[Johnix1337]

Hey.
I want to give you some tips with your code.

class User{
    public function database($h, $u, $p, $db){
        $this->conn = new mysqli($h,$u,$p,$db); // So i don't have to use global.
    }

public function grabData($k) {
    $q = $this->conn->query("SELECT * FROM users WHERE username = '$this->session'");
    $r = mysqli_fetch_assoc($q);

    return $r[$k];
}
} 

The grabData() method is pretty inefficient. I would store the result which is given in a private attribute, so it doesn't have to run the query everytime grabData is called.
Another tip is, that you use dependency injection with the database. The method database() will create an mysqli instance everytime when you use the User object.
I would pass the Database as a parameter so the database() method looks a bit like this.

class User{
/* ... */
public function database(MySQLi $connection)
{
this->conn = $connection;
} 

And if you want to call this function, you simply do this:

$connection = new MySQLi('bla','bla','bla','bla');

$user = new User('blabla');
$user->database($connection); 

I would put the $connection variable in the core file, so you only use one instance of it.

I've rewritten the User class with minor adjustments:

class User {
    
    private $row = null;
    
    public function __construct($session) {
        $this->session = $session;
    }
    
    public function database($connection){
        $this->conn = $connection; // Dependency Injection, maybe put this in the constructor
    }

    public function grabData($k) {
        if (is_null($this->row)){
        $q = $this->conn->query("SELECT * FROM users WHERE username = '$this->session'");
        $this->row = mysqli_fetch_assoc($q);
        }
        
        return $this->row[$k];
    }
    
    // Call RefreshRow() if something has been updated in the database
    public function RefreshRow()
    {
        $q = $this->conn->query("SELECT * FROM users WHERE username = '$this->session'");
        $this->row = mysqli_fetch_assoc($q);
    }
} 

I didnt't test anything so forgive me if I made any mistakes.
Hope I could help!

[zSanta]

It looks really good. Keep up the good work!


Verzonden vanaf mijn iPhone met behulp van Tapatalk

[Even Retro]

I got a white client, how do i fix it? Plz reply fast!

[Emily]

$q = $this->conn->query("SELECT * FROM users WHERE username = '$this->session'");

This is where prepared statements are for...

Also, I hate the style (no offence) but it looks TOO small in my opinion and really messy. I don't know much about your PHP but like above: that's what prepared statements are for..

Won't use it as I use my own custom CMS but still good attempt.

[Finley7]

$q = $this->conn->query("SELECT * FROM users WHERE username = '$this->session'");

This is where prepared statements are for...

Also, I hate the style (no offence) but it looks TOO small in my opinion and really messy. I don't know much about your PHP but like above: that's what prepared statements are for..

Won't use it as I use my own custom CMS but still good attempt.

I used them, but i tought that they didn't work. So i replaced them with normal querys. I will try to update them along the month. And i don't like the style either. I made it so the cms wasn't empty. But im busy on an improved design with a talented designer. No worries!

Sent from my GT-I8260 using Tapatalk

- - - Updated - - -

So there was another sso problem. Fixed it again:
https://mega.co.nz/#!Y0IwGCzS!LGSyWVo1s8p80FcDYl85iBZ3TCUz5RzyXnI4WkMD_7g

[thatsronnie4u]

Design is good, well done!

[Volx]

[Finley7]

If it does't work, use any of these dbs: phoenix, swift and bc storm

[Gaby]

Finley is so easy to be trolled. :$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$ ahum ahum

OT: Well done ;)