Changeset 2451

Show
Ignore:
Timestamp:
09/05/06 09:17:50 (6 months ago)
Author:
demian
Message:

completed work for basic comment functionality

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • branches/0.6-bugfix/CHANGELOG.txt

    r2441 r2451  
    1717 
    1818IMPROVEMENTS 
     1905-08-06    Added getCurrentUrl() to Output object so <a href="{getCurrentUrl()}">foo</a> 
     20            in a template will create a link to the currently requested page 
    192104-08-06    Deprecated DA_Default::moduleIsRegistered() in favour of SGL::moduleIsEnabled() 
    202203-08-06    Upgraded PEAR libs including HTML_Ajax, note: you now have to call DAOs 
  • branches/0.6-bugfix/lib/SGL/Inflector.php

    r2447 r2451  
    131131            $name .= 'Mgr'; 
    132132        } 
    133         return SGL_Inflector::caseFix(ucfirst($name)); 
     133        return SGL_Inflector::caseFix(ucfirst($name), $force = true); 
    134134    } 
    135135 
     
    203203        $aConfValuesLowerCase = array_map('strtolower', $aConfValues); 
    204204        $isFound = array_search(strtolower($str), $aConfValuesLowerCase); 
     205        //  iterate through unloaded config files until key is found (not implemented) 
     206        if (!$isFound) { 
     207            #crude hack, avoid mgrs with interior bumpy caps 
     208            if ($str == 'ArticleviewMgr') { 
     209                $str = 'ArticleViewMgr'; 
     210            } 
     211        } 
    205212        return ($isFound !== false) ? $aConfValues[$isFound] : $str; 
    206213    } 
  • branches/0.6-bugfix/lib/SGL/Output.php

    r2447 r2451  
    557557    } 
    558558 
     559    function getCurrentUrl() 
     560    { 
     561        $reg =& SGL_Registry::singleton(); 
     562        $oCurrentUrl = $reg->getCurrentUrl(); 
     563        return $oCurrentUrl->toString(); 
     564    } 
     565 
    559566    function isVerticalNav($styleSheet) 
    560567    { 
     
    623630        $frmCallerMgr = $req->get('frmCallerMgr'); 
    624631        $mgrName = (is_null($frmCallerMgr)) 
    625             ? $req->getModuleName() 
     632            ? $req->getManagerName() 
    626633            : $frmCallerMgr; 
    627634        return $mgrName; 
  • branches/0.6-bugfix/modules/comment/classes/CommentMgr.php

    r2449 r2451  
    7373        $input->callerMod   = $req->get('frmCallerMod'); 
    7474        $input->callerMgr   = $req->get('frmCallerMgr'); 
     75        $input->callerId    = $req->get('frmCallerId'); 
    7576        $input->callerTmpl  = $req->get('frmCallerTmpl'); 
    7677 
     
    112113        $oComment->comment_id = $this->dbh->nextId('comment'); 
    113114        $oComment->date_created = SGL_Date::getTime(true); 
     115        if (!is_null($input->callerId)) { 
     116            $oComment->entity_id = $input->callerId; 
     117        } 
     118        $oComment->type = 'comment'; 
     119        //  get remote IP 
    114120        $success = $oComment->insert(); 
    115121 
     
    126132        SGL::logMessage(null, PEAR_LOG_DEBUG); 
    127133 
    128         SGL_HTTP::redirect(array( 
     134        $aRedirect = array( 
    129135            'moduleName'  => $input->callerMod, 
    130             'managerName' => $input->callerMgr) 
    131             ); 
     136            'managerName' => $input->callerMgr); 
     137        if (!is_null($input->callerId)) { 
     138            $aRedirect['frmArticleID'] = $input->callerId; 
     139        } 
     140 
     141        SGL_HTTP::redirect($aRedirect); 
    132142    } 
    133143} 
  • branches/0.6-bugfix/modules/default/templates/comments.html

    r2450 r2451  
    99        <a name="comment{increment(key)}"></a> 
    1010        <!--COMMENT PERMALINK--> 
    11         <a href="{makeUrl()}#comment{increment(key)}">#{increment(key)}</a> 
     11        <a href="{getCurrentUrl()}#comment{increment(key)}">#{increment(key)}</a> 
    1212        <!--AUTHOR--> 
    1313        {if:oComment.url} 
     
    3131<form action="{makeUrl(#insert#,#comment#,#comment#)}" method="post" flexy:ignore> 
    3232    <fieldset class="hide"> 
    33         <input type="hidden" name="comment[entity_name]" value="faq" /> 
    34         <input type="hidden" name="frmCallerMod" value="{getCurrentModule()}" /> 
    35         <input type="hidden" name="frmCallerMgr" value="{getCurrentManager()}" /> 
    36         <input type="hidden" name="frmCallerTmpl" value="{getCurrentTemplate()}" /> 
     33        <input type="hidden" name="comment[entity_name]" value="{getCurrentManager()}" /> 
     34        <input type="hidden" name="frmCallerMod"    value="{getCurrentModule()}" /> 
     35        <input type="hidden" name="frmCallerMgr"    value="{getCurrentManager()}" /> 
     36        <input type="hidden" name="frmCallerId"     value="{articleID}" /> 
     37        <input type="hidden" name="frmCallerTmpl"   value="{getCurrentTemplate()}" /> 
    3738    </fieldset> 
    3839 
     
    6364            </dd> 
    6465 
    65             <dt><label for="comment[notify]"> </label></dt> 
     66            <!--<dt><label for="comment[notify]"> </label></dt> 
    6667            <dd> 
    6768                <input type="checkbox" name="comment[notify]" id="comment[notify]" value="{comment.notify}" /> 
    6869                E-mail notification 
    69             </dd> 
     70            </dd>--> 
    7071 
    7172            <dt> </dt> 
    7273            <dd> 
    73                 <input type="submit" name="submitted" value="preview" /
    74                 <input type="submit" name="submitted" value="post" /> 
     74                <!--<input type="submit" name="submitted" value="preview" />--
     75                <input class="wideButton" type="submit" name="submitted" value="post" /> 
    7576            </dd> 
    7677        </dl> 
  • branches/0.6-bugfix/modules/faq/conf.ini

    r2441 r2451  
    11[FaqMgr] 
    22requiresAuth    = false 
    3 commentsEnabled = fals
     3commentsEnabled = tru
    44 
    55[AdminFaqMgr] 
  • branches/0.6-bugfix/modules/publisher/classes/ArticleViewMgr.php

    r2425 r2451  
    6666            'summary'   => array('summary'), 
    6767        ); 
     68        //  enable comments if configured 
     69        if (SGL::moduleIsEnabled('comment')) { 
     70            require_once SGL_MOD_DIR  . '/comment/classes/CommentDAO.php'; 
     71            require_once SGL_CORE_DIR . '/Delegator.php'; 
     72            $dao = &CommentDAO::singleton(); 
     73            $this->da  = new SGL_Delegator(); 
     74            $this->da->add($dao); 
     75        } 
    6876    } 
    6977 
     
    7684        $input->masterTemplate  = $this->masterTemplate; 
    7785        $input->template        = $this->template; 
    78   
     86 
    7987        //  form vars 
    8088        $input->action          = ($req->get('action')) ? $req->get('action') : 'view'; 
     
    138146            $output->staticArticle = true; 
    139147        } 
     148        //  display comments? 
     149        if (!empty($this->conf['ArticleViewMgr']['commentsEnabled'])) { 
     150            $output->aComments = $this->da->getCommentsByEntityId('articleview'); 
     151        } 
    140152    } 
    141153 
  • branches/0.6-bugfix/modules/publisher/conf.ini

    r2371 r2451  
    11[ArticleViewMgr] 
    22requiresAuth    = false 
     3commentsEnabled = false 
    34 
    45[ArticleMgr] 
     
    1516formatFileName  = true 
    1617 
    17 ;note: you can just remove file types.  
     18;note: you can just remove file types. 
    1819;for adding new stuff you have also to modify DocumentMgr.php 
    1920allowedFileTypes = gif,jpeg,jpg,png,doc,rtf,txt,pdf,ppt,pps,xls,htm,html,csv 
  • branches/0.6-bugfix/modules/publisher/templates/articleView.html

    r2333 r2451  
    1 <h1 class="pageTitle">{leadArticle[title]}</h1> 
     1<h1 flexy:if="leadArticle[title]" class="pageTitle">{leadArticle[title]}</h1> 
     2<h1 flexy:if="!leadArticle[title]" class="pageTitle">{translate(pageTitle)}</h1> 
    23<div class="message">{msgGet()}</div> 
    34 
    45{if:staticArticle} 
    56    <div id="staticArticle"> 
    6         {if:leadArticle}  
     7        {if:leadArticle} 
    78            {t:articleOutput} 
    89        {else:} 
    9             {translate(#No lead article found#)} 
     10            <p flexy:if="leadArticle[title]">{translate(#No lead article found#)}</p> 
    1011        {end:} 
    1112    </div> 
    1213{else:} 
    1314<!-- If not a static article display option blocks --> 
    14 <h2 class="sectionHeader">{translate(#Current Category#)}: {currentCat}</h2> 
    15 <h3 class="colHeader">{translate(#You are here#)}: {path:h}</h3> 
    16     <div id="options"> 
     15<h2 flexy:if="currentCat" class="sectionHeader">{translate(#Current Category#)}: {currentCat}</h2> 
     16<h3 flexy:if="path" class="colHeader">{translate(#You are here#)}: {path:h}</h3> 
     17    <div flexy:if="leadArticle[title]" id="options"> 
     18        {if:articleList} 
    1719        <div class="options-block"> 
    1820            <div class="blockHeader">{translate(#Related Articles#)}</div> 
    1921                <div class="blockContent"> 
    20             {if:articleList} 
    2122                <ul class="noindent"> 
    2223                {foreach:articleList,key,aValue} 
     
    2425                {end:} 
    2526                </ul> 
    26             {else:} 
    27                 {translate(#No articles found#)} 
    28             {end:} 
    2927            </div> 
    3028        </div> 
    31      
     29        {end:} 
     30 
    3231        {if:documentList} 
    3332        <div class="options-block"> 
     
    4342        </div> 
    4443        {end:} 
    45     </div> 
     44 
     45    </div><!--END options--> 
    4646 
    4747    <div id="leadArticle"> 
    48         {if:leadArticle}  
     48        {if:leadArticle} 
    4949            {t:articleOutput} 
    5050        {else:} 
    51             {translate(#No lead article found#)} 
     51            <p flexy:if="leadArticle[title]">{translate(#No lead article found#)}</p> 
    5252        {end:} 
    5353    </div> 
    5454{end:} 
     55 
     56<!-- ARE COMMENTS ENABLED --> 
     57{if:conf[ArticleViewMgr][commentsEnabled]} 
     58    <flexy:include src="comments.html" /> 
     59{end:} 
  • branches/0.6-bugfix/www/themes/default/css/core.php

    r2443 r2451  
    5454} 
    5555 
     56hr { 
     57    border-top: 1px dotted #999; 
     58    border-bottom: 0px; 
     59    height: 1px; 
     60} 
     61 
    5662/* 
    5763======================Global layaout==========================*/ 
     
    399405    padding: 10px; 
    400406    border: 1px solid grey; 
    401 } 
    402 #addComment p label { 
    403     width: 5em; 
    404     float: left; 
    405     text-align: right; 
    406     margin-right: 0.5em; 
    407     display: block 
    408407} 
    409408#addComment input[type="text"] {