PHP Retro Style MVC Framework?

[CoderMaya]

Oh and -
http://code.google.com/p/retrophpmvc/

[CoderMaya]

arkinstall - the dump function calls var_dump wrapped with PRE tags, so yes unfortunately it outputs them.

If it's a community project, though, I'm sure at some point someone would love to replace that.

[Jake Arkinstall]

Waiting to be added to the project, but heres a suggestion for the returning a variable:

function _dump()
{
    $args = func_get_args();
    ob_start();
    var_dump($args);
    return sprintf('<pre>%s</pre>', ob_get_clean());
} 

[CoderMaya]

Haha, obviously. I'll add it right now.

[CoderMaya]

Working demo:

class Posts extends Controller
{
    public function index($request)
    {
        $posts = find('posts');
        // Add a relationship and get all the corresponding records
        // The neat thing is that every record you get is not read-only
        // you can set new values and save() it.
        $posts = relate($posts, 'id', 'comments', 'post_id');
        $data['posts'] = $posts;
        render('posts.index', $data);
    }
    
    public function update($request)
    {
        // Find a single comment based on filters and update it
        $c = single('comments', array('post_id' => 2));
        $c->text = 'the only comment for this post';
        $c->save();
    }
    
    public function add($request)
    {
        // Was the form sent?
        if($request->isPostBack)
        {
            // Create a new comment
            $post = record('comments');
            $post->post_id = post('post_id');
            $post->text = post('subject');
            $post->save();
        }
        render('posts.add');
    }
} 

Show posts template:

<? foreach($posts->loop() as $post):  ?>

<h1><?=html($post->subject)?> at <?=dt2d($post->date, 'd/m/y')?></h1>

    <? foreach($post['comments']->loop() as $comment):  ?>
    <p><?=html($comment->text)?></p>
    <? endforeach; ?>

<? endforeach; ?>

[Jake Arkinstall]

Cool.

Just wondering, you have a few functions which do similar things, such as:

function redirect($url)
{
    if(!preg_match('~^http://~', $url))
    {
        $url = $GLOBALS['rooturi'].$url;
    }

    header('Location: '.$url);
}

/* reload the current page */
function reload()
{
    $url = 'http://'.$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'];
    header('Location: '.$url);
} 

Surely it would be better:

function redirect($url)
{
    if(!preg_match('~^http://~', $url))
    {
        $url = $GLOBALS['rooturi'].$url;
    }

    header('Location: '.$url);
}

/* reload the current page */
function reload()
{
    $url = 'http://'.$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'];
    redirect($url);
} 

(ooh, and before I forget - put exit; after using header('location....') to stop page processing afterwards).

Edit:

Update to redirect():

function redirect($url)
{
    if(!preg_match('~^https?://~', $url))
    {
        $url = $GLOBALS['rooturi'].$url;
    }
    header('Location: '.$url);
    exit;
} 

[CoderMaya]

Yeah of course. I kind of rushed through a lot of it so it's good you let me know about these small errors in design.

I also made a video... Uploading now, will post when it's done.

BTW Jake, I PMed you.

[CoderMaya]

Ok the video is just a tad annoying because the noise removal feature had some funny side-effects and the frame rate is 4 but those are just minor things, it's a good video introduction to the framework.

It shows how to work with the database and views and you can see the results "live".

http://serialize.us/retro/retro.html

[Jake Arkinstall]

Everything is so much simpler like that...

Round of applause!

[CoderMaya]

Thank you, thank you. I actually came up with the idea last week.. And I don't need to tell you how it is when you think you have a good idea, right? Haha... Yeah, couldn't wait to get it to work.

Jake, if you didn't still get my PM - add me to MSN - yam.marc@hotmail.com so we can work on this more closely.

All the rest - again - feel free to add me if you want to help out!

[Czaries]

Nice video... is there a way to turn off the phoney keyboard typing sounds when you're recording?

[CoderMaya]

Yeah, should I next time?

[snajt]

I'm definitely interesting in helping you out with development on this, just watched your video and it looks great from what I've seen. I'm going to test it more thoroughly wen i get home from work, keep up the good work man!

[World Wide Weird]

I just looked at the project upload times - 14 minutes between builds. You guys are really jamming! I'm going to have a look at the code and see if I have any ideas to contribute.

[Renard Urbain]

Sent you a mail Marc this sounds really interesting !

[Jake Arkinstall]

Latest build out.

Seriously, this thing is getting great!

[CoderMaya]

http://code.google.com/p/retrophpmvc/downloads/list

Jake also covered up the functions we had in build 2 so far, thanks much! Me and Jake are working well together and just think if we could all get into a group (Jake suggested a sitepoint one, perhaps?) or MSN convos and brain storm until we get exactly what we want!

[CoderMaya]

I also think the secret of this framework is:

We're not looking for ways to make it too clever or complex or flexible.

We're admitting that we are lazy programmers, and lazy development is what we want! And I think most people are like that, and just didn't want to be the first to put it out there.

[World Wide Weird]

Latest build out.

Seriously, this thing is getting great!

Whoa! I haven't even had a chance to open the Zip file for build 3 yet - I'm still watching the video!

[kyberfabrikken]

I noticed that there are a lot of places, where you're embedding strings in HTML, without escaping properly. Attributes needs to be escaped with htmlspecialchars.

Eg. instead of this:

return '<img src="'.$link.'" alt="'.$link.'" />'; 

You should do this:

return '<img src="'.htmlspecialchars($link).'" alt="'.htmlspecialchars($link).'" />'; 

[Czaries]

Are you guys not working in SVN? I don't see anything in the trunk and the changelog is empty. You guys really need to get this thing in Google's SVN repository so that changes can be tracked and viewed online without downloading the package.

[Jake Arkinstall]

Yeah - we will do.

The current isn't exactly a version yet. He just posted it so that people could see the files at the moment - it still has alot to go before being ready for versioning

[webaddictz]

Yeah - we will do.

The current isn't exactly a version yet. He just posted it so that people could see the files at the moment - it still has alot to go before being ready for versioning

As I just said to Yam: versioning is really usefull for development, not only for releasing. It makes for a good platform for collaboration. Now, the way I see it there are more people who are willing to contribute, so I'd recommend reconsidering that choice.

[Jake Arkinstall]

Cool - we will do after this next stage then.

We're just making some, well, nothing better than kickass! form handling, so after this stage we'll go ahead with the versioning.

[CoderMaya]

I noticed that there are a lot of places, where you're embedding strings in HTML, without escaping properly. Attributes needs to be escaped with htmlspecialchars.

Eg. instead of this:

return '<img src="'.$link.'" alt="'.$link.'" />'; 

You should do this:

return '<img src="'.htmlspecialchars($link).'" alt="'.htmlspecialchars($link).'" />'; 

Good point. I'll fix that tomorrow, probably.

Anyway people - form validation is officially completed.

Story:

Me and Jake were chatting about how to go through with it... we came up with a pretty nice but vague idea - I went for a walk outside for an hour, came back with motivation... result:

if($request->isPostBack)
        {
            post('username')->min_chars(4)->max_chars(6);
            post('email')->email();
            post('author')->min_words(2);
            post('anything')->match('^pattern$');
            post('age')->ints();
            
            if(post_valid())
                echo 'yay';
            else
                echo 'boo';
                
            // or
            if(post('email')->email()->is_valid())
                echo 'yay';
            else
                echo 'boo';
        } 

It also works with array posts such as <input type="text" name="names[]" />

Build 5 is up in http://code.google.com/p/retrophpmvc/downloads/list - go ahead and beta test it.