Brand new template and redesign please review!

Discussion in 'Websites' started by bbrian017, Sep 28, 2008.

  1. #1
    So it's been a year Blog Engage has been up and active! I thought a new template would help the site and members have a fresh new look at it all! I think it's very 2.0 web style!

    If you have Google tool bar the cache is still the old site and you can see what it looked like before!

    Over all what do you think is it looking better?

    Social Networking for Bloggers

    Cheers!
     
    bbrian017, Sep 28, 2008 IP
  2. skypeace

    skypeace Well-Known Member

    Messages:
    2,348
    Likes Received:
    125
    Best Answers:
    0
    Trophy Points:
    130
    #2
    Firstly, I love the domain name :)

    The favicon is good but the logo is very unimpressive. ( Let me know if you need one :) )

    The colors really rock :)

    The ads totally screw up the page sadly.

    It may be just me, but i feel there must be more space between forgot password and no account.

    Hope i helped! :)
     
    skypeace, Sep 28, 2008 IP
    bbrian017 likes this.
  3. bbrian017

    bbrian017 Well-Known Member

    Messages:
    2,990
    Likes Received:
    66
    Best Answers:
    0
    Trophy Points:
    170
    #3
    I agree with you 100%

    This template sucks for ads as opposed to my old one!

    The banner was 10 usd and very fast work! This was a decision from me and low cost was being associated!

    I tried editing that sill spot removing register all together but when I do I get an error after logging in!

    	</div>
    	<div class="box login_footer">
    		<a href="{$URL_login}#forgotten">Forgotten Password?</a>
    		<a href="{$URL_register}">No Account? Register Now!</a>
    	</div>
    Code (markup):
    All I remove is this,

    <a href="{$URL_register}">No Account? Register Now!</a>
    Code (markup):
    The error I get s as follows!

    <?php
    // The source code packaged with this file is Free Software, Copyright (C) 2005 by
    // Ricardo Galli <gallir at uib dot es>.
    // It's licensed under the AFFERO GENERAL PUBLIC LICENSE unless stated otherwise.
    // You can get copies of the licenses here:
    //         http://www.affero.org/oagpl.html
    // AFFERO GENERAL PUBLIC LICENSE is also included in the file called "COPYING".
    
    include_once('Smarty.class.php');
    $main_smarty = new Smarty;
    
    include('config.php');
    include(mnminclude.'html1.php');
    include(mnminclude.'link.php');
    include(mnminclude.'smartyvariables.php');
    
    // breadcrumbs and page title
    $navwhere['text1'] = $main_smarty->get_config_vars('PLIGG_Visual_Breadcrumb_Login');
    $navwhere['link1'] = getmyurl('loginNoVar', '');
    $main_smarty->assign('navbar_where', $navwhere);
    $main_smarty->assign('posttitle', " / " . $main_smarty->get_config_vars('PLIGG_Visual_Breadcrumb_Login'));
    
    // sidebar
    $main_smarty = do_sidebar($main_smarty);
    
    // initialize error message variable
    $errorMsg="";
    
    // if user requests to logout
    if(isset($_GET["op"])){
        if($_GET["op"] === 'logout') {
            $current_user->Logout(sanitize($_REQUEST['return'], 3));
        }
    }
    
    // if user tries to log in
    if(isset($_POST["processlogin"]) || isset($_GET["processlogin"])){
        if($_POST["processlogin"] == 1) { // users logs in with username and password
            $username = sanitize(trim($_POST['username']), 3);
            $password = sanitize(trim($_POST['password']), 3);
            if(isset($_POST['persistent'])){$persistent = $_POST['persistent'];}else{$persistent = '';}
            if($current_user->Authenticate($username, $password, $persistent) == false) {
                $errorMsg=$main_smarty->get_config_vars('PLIGG_Visual_Login_Error');
            } else {
                if(strlen(sanitize($_REQUEST['return'], 3)) > 1) {
                    $return = sanitize($_REQUEST['return'], 3);
                } else {
                    $return =  my_pligg_base.'/';
                }
                
                define('logindetails', $username . ";" . $password . ";" . $return);
    
                $vars = '';
                check_actions('login_success_pre_redirect', $vars);
    
                if(strpos($_SERVER['SERVER_SOFTWARE'], "IIS") && strpos(php_sapi_name(), "cgi") >= 0){
                    echo '<SCRIPT LANGUAGE="JavaScript">window.location="' . $return . '";</script>';
                    echo $main_smarty->get_config_vars('PLIGG_Visual_IIS_Logged_In') . '<a href = "'.$return.'">' . $main_smarty->get_config_vars('PLIGG_Visual_IIS_Continue') . '</a>';
                } else {
                    header('Location: '.$return);
                }
                die;
            }
        }
    
        if($_POST["processlogin"] == 3) { // if user requests forgotten password
            $username = $db->escape(trim($_POST['username']));
            if(strlen($username) == 0){
                $errorMsg = $main_smarty->get_config_vars("PLIGG_Visual_Login_Forgot_Error");
            }
            else {
                $user = $db->get_row("SELECT * FROM `" . table_users . "` where `user_login` = '".$username."'");
                if($user){
                    $salt = substr(md5(uniqid(rand(), true)), 0, SALT_LENGTH);
                    $saltedlogin = generateHash($user->user_login);
    
                    $to = $user->user_email;
                    $subject = $main_smarty->get_config_vars("PLIGG_PassEmail_Subject");
                    $body = $main_smarty->get_config_vars("PLIGG_PassEmail_Body") . $my_base_url . $my_pligg_base . '/login.php?processlogin=4&username=' . $username . '&confirmationcode=' . $saltedlogin;
                    $headers = 'From: ' . $main_smarty->get_config_vars("PLIGG_PassEmail_From") . "\r\n";
    
    
                    if(time() - strtotime($user->last_reset_request) > $main_smarty->get_config_vars("PLIGG_PassEmail_LimitPerSecond")){
                        if (mail($to, $subject, $body, $headers))
                        {
                            $main_smarty->assign('user_login', $user->user_login);
                            $main_smarty->assign('profile_url', getmyurl('profile'));
                            $main_smarty->assign('login_url', getmyurl('loginNoVar'));
    
                            $errorMsg = $main_smarty->get_config_vars("PLIGG_PassEmail_SendSuccess");
    
                            $db->query('UPDATE `' . table_users . '` SET `last_reset_code` = "'. $saltedlogin . '" WHERE `user_login` = "'.$username.'"');
                            $db->query('UPDATE `' . table_users . '` SET `last_reset_request` = FROM_UNIXTIME('.time().') WHERE `user_login` = "'.$username.'"');
                            
                            define('pagename', 'login'); 
                          $main_smarty->assign('pagename', pagename);
    
                        }
                        else
                        {
                            $errorMsg = $main_smarty->get_config_vars('PLIGG_Visual_Login_Delivery_Failed');
                        }
                    }
                    else{
                        $errorMsg = $main_smarty->get_config_vars("PLIGG_PassEmail_LimitPerSecond_Message");
                    }
                }
                else{
                    $errorMsg = $main_smarty->get_config_vars('PLIGG_Visual_Login_Does_Not_Exist');
                }
            }
        }
    
        if($_GET["processlogin"] == 4) { // if user clicks on the forgotten password confirmation code
            $username = sanitize(trim($_GET['username']), 3);
            if(strlen($username) == 0){
                $errorMsg = $main_smarty->get_config_vars("PLIGG_Visual_Login_Forgot_Error");
            }
            else {
                $confirmationcode = $_GET["confirmationcode"];
                $DBconf = $db->get_var("SELECT `last_reset_code` FROM `" . table_users . "` where `user_login` = '".$username."'");
                if($DBconf){
                    if($DBconf == $confirmationcode && !empty($confirmationcode)){
                        $db->query('UPDATE `' . table_users . '` SET `last_reset_code` = "" WHERE `user_login` = "'.$username.'"');
                        $db->query('UPDATE `' . table_users . '` SET `user_pass` = "033700e5a7759d0663e33b18d6ca0dc2b572c20031b575750" WHERE `user_login` = "'.$username.'"');
                        $errorMsg = $main_smarty->get_config_vars('PLIGG_Visual_Login_Forgot_PassReset');
                    }    else {
                        $errorMsg = $main_smarty->get_config_vars('PLIGG_Visual_Login_Forgot_ErrorBadCode');
                    }
                } else {
                    $errorMsg = $main_smarty->get_config_vars('PLIGG_Visual_Login_Forgot_ErrorBadCode');
                } 
            }
        }
    }   
        
    // pagename
    define('pagename', 'login'); 
    $main_smarty->assign('pagename', pagename);
     
    // misc smarty 
    $main_smarty->assign('errorMsg',$errorMsg);  
    $main_smarty->assign('register_url', getmyurl('register'));
    
    // show the template
    $main_smarty->assign('tpl_center', $the_template . '/login_center');
    $main_smarty->display($the_template . '/pligg.tpl');
    
    ?>
    Code (markup):
    Line 60 is here,

    header('Location: '.$return);
    Code (markup):
    I'm still working on a few ideals and things are still in beta!

    Thanks for the awesome review!
     
    bbrian017, Sep 28, 2008 IP
    farrhad likes this.
  4. skypeace

    skypeace Well-Known Member

    Messages:
    2,348
    Likes Received:
    125
    Best Answers:
    0
    Trophy Points:
    130
    #4
    skypeace, Sep 28, 2008 IP
  5. Ecreation

    Ecreation Active Member

    Messages:
    823
    Likes Received:
    26
    Best Answers:
    0
    Trophy Points:
    68
    #5
    Your site looks nice.
     
    Ecreation, Sep 28, 2008 IP
  6. kingofsanda

    kingofsanda Peon

    Messages:
    8,154
    Likes Received:
    218
    Best Answers:
    0
    Trophy Points:
    0
    #6
    I think the site looks good, the theme, the color scheme, the page layout,
    and the ad placement looks good.
     
    kingofsanda, Sep 28, 2008 IP
    bbrian017 likes this.
  7. DiEgO_VC

    DiEgO_VC Peon

    Messages:
    158
    Likes Received:
    5
    Best Answers:
    0
    Trophy Points:
    0
    #7
    Looking good! the layout is great and the site seems to be very complete
     
    DiEgO_VC, Sep 28, 2008 IP
  8. bbrian017

    bbrian017 Well-Known Member

    Messages:
    2,990
    Likes Received:
    66
    Best Answers:
    0
    Trophy Points:
    170
    #8
    Could anyone possibly make some comments on articlea and see if you have issue with the reply not working?

    Sometimes when I leave a reply it doesn't stay but no one else can confirm this issue but me!

    Rep can be added for those who reply to some comments and provide feedback!
     
    bbrian017, Sep 29, 2008 IP