Changeset 6006


Ignore:
Timestamp:
02/05/12 18:56:03 (4 months ago)
Author:
vain
Message:
  • cleanups, removed splitting table rows
Location:
trunk/core
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/core/errorhandler.core.php

    r5998 r6006  
    310310        $errormessage .= '</table>'; 
    311311 
    312         # HR Split 
    313         $errormessage .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    314  
    315312        # Error Context 
    316         $errormessage .= '<tr><td colspan="2"><h3>Context</h3></td></tr>'; 
    317         $errormessage .= '<tr><td colspan="2">' . self::getErrorContext($errorfile, $errorline, 8) . '</td></tr>'; 
    318  
    319         # HR Split 
    320         $errormessage .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
     313        $errormessage .= '<tr><td colspan="2"><h3>Context</h3>'; 
     314        $errormessage .= self::getErrorContext($errorfile, $errorline, 8) . '</td></tr>'; 
    321315 
    322316        # Add Debug Backtracing 
    323317        $errormessage .= '<tr><td>' . self::getDebugBacktrace($trimed_errorstring) . '</td></tr>'; 
    324318 
    325         # HR Split 
    326         # $errormessage .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    327  
    328319        # 
    329320        # $errormessage .= '<tr><td>' . self::getBugtrackerSearch() . '</td></tr>'; 
    330  
    331         # HR Split 
    332         $errormessage .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    333321 
    334322        # Environmental Informations at Errortime ( $errorcontext is not displayed ) 
    … …  
    345333        $errormessage .= '</table>'; 
    346334 
    347         # HR Split 
    348         $errormessage .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    349  
    350335        # Backlink to Bugtracker with Errormessage -> http://trac.clansuite.com/newticket 
    351336        $errormessage .= self::getBugtrackerMessage($errorstring, $errorfile, $errorline, $errorcontext); 
    … …  
    400385        # prepare a new backtrace_string 
    401386        $backtrace_string = ''; 
    402         $backtrace_string .= '<tr><td><h3>Backtrace</h3></td></tr>'; 
    403         $backtrace_string .= '<tr><td width="95%">'; 
     387        $backtrace_string .= '<tr><td width="95%"><h3>Backtrace</h3>'; 
    404388        $backtrace_string .= '<table class="cs-backtrace-table" width="95%">'; 
    405         $backtrace_string .= '<tr><th><strong>Callstack</strong></td><th colspan="2">(Recent function calls last)</td></tr>'; 
    406  
    407         $backtrace_string .= '<tr><th width="2%">#</th><th>Function</th><th width="40%">Location</th></tr>'; 
     389 
     390        # table row 1 - header 
     391        $backtrace_string .= '<tr><th width="2%">Callstack</th><th>Function (recent function calls last)</th><th width="40%">Location</th></tr>'; 
    408392 
    409393        $backtraces_count = count($backtrace)-1; 

  • trunk/core/exception.core.php

    r6001 r6006  
    265265    public function yellowScreenOfDeath() 
    266266    { 
     267        /** 
     268         * @todo add backlink to the exception codes list 
     269         */ 
     270        if($this->code > 0) 
     271        { 
     272            $code = '(# ' . $this->code . ')'; 
     273        } 
     274        else 
     275        { 
     276            $code = ''; 
     277        } 
     278 
    267279        # Header 
    268280        $errormessage = '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'; 
    269281        $errormessage .= '<html xmlns="http://www.w3.org/1999/xhtml" lang="en" xml:lang="en">'; 
    270282        $errormessage .= '<head>'; 
    271         $errormessage .= '<title>Clansuite Exception : [ ' . $this->message . ' | Exceptioncode: ' . $this->code . ' ] </title>'; 
     283        $errormessage .= '<title>Clansuite Exception ' . $code . ' - ' . $this->message . '</title>'; 
    272284        $errormessage .= '<link rel="stylesheet" href="' . WWW_ROOT_THEMES_CORE . 'css/error.css" type="text/css" />'; 
    273285        $errormessage .= '</head>'; 
    … …  
    285297 
    286298        # Fieldset Legend 
    287         $errormessage .= '<legend>Clansuite Exception : [ ' . $this->message . ' ]</legend>'; 
     299        $errormessage .= '<legend>Clansuite Exception</legend>'; 
    288300 
    289301        # Error Messages from Object (table) 
    … …  
    291303        $errormessage   .= '<table>'; 
    292304 
    293         # @todo add link 
    294         if($this->code > 0) 
    295         { 
    296             $code = '(# ' . $this->code . ')'; 
    297         } 
    298         else 
    299         { 
    300             $code = ''; 
    301         } 
    302  
    303         $errormessage   .= '<tr><td>'; 
    304  
    305         # Panel 1 - Exception Message and File 
     305        /** 
     306         * Panel 1 
     307         * 
     308         * Exception Message and File 
     309         */ 
     310        $errormessage  .= '<tr><td>'; 
    306311        $errormessage   .= '<div id="panel"'; 
    307312        $errormessage   .= 'style="background: none repeat scroll 0 0 #FDF599; border: 1px solid #E03937; border-radius: 3px; padding: 0 10px 10px;">'; 
    … …  
    311316        $errormessage   .= '&nbsp;<strong>Line: </strong>'.$this->line; 
    312317        $errormessage   .= '</div>'; 
    313  
    314318        $errormessage   .= '</td></tr>'; 
    315319 
    316         # Split 
    317         $errormessage   .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    318  
    319         # Debug Backtrace 
     320        /** 
     321         * Panel 2 
     322         * 
     323         * Debug Backtrace 
     324         */ 
    320325        if ( defined('DEBUG') and DEBUG == 1 ) 
    321326        { 
    322327            # lets get the backtrace as html table 
    323328            $errormessage   .= Clansuite_Errorhandler::getDebugBacktrace($this->trace); 
    324  
    325             # Split 
    326             $errormessage   .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    327         } 
    328  
    329         # Environmental Informations at Errortime 
     329        } 
     330 
     331        /** 
     332         * Panel 3 
     333         * 
     334         * Server Environment Informations 
     335         */ 
    330336        if ( defined('DEBUG') and DEBUG == 1 ) 
    331337        { 
    332             # HEADING <Server Environment> 
    333338            $errormessage .= '<tr><td><table width="95%">'; 
    334339            $errormessage .= '<tr><td colspan="2"><h3>Server Environment</h3></td></tr>'; 
    … …  
    342347            $errormessage .= ' (' . CLANSUITE_VERSION_NAME . ') [Revision #' . CLANSUITE_REVISION . ']</td></tr>'; 
    343348            $errormessage .= '</table></td></tr>'; 
    344  
    345             # Split 
    346             $errormessage .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    347         } 
    348  
    349         # HEADING <Additional Information> 
     349        } 
     350 
     351        /** 
     352         * Panel 4 
     353         * 
     354         * Additional Information 
     355         */ 
    350356        if(empty(self::$exception_template) === false) 
    351357        { 
    352             $errormessage  .= '<tr><td colspan="2"><h3>Additional Information & Solution Suggestion</h3></td></tr>'; 
    353             $errormessage  .= '<tr><td colspan="2">'.self::$exception_template.'</td></tr>'; 
    354  
    355             # Split 
    356             $errormessage  .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    357         } 
    358  
    359         # HEADING <Rapid Development> 
    360  
     358            $errormessage  .= '<tr><td colspan="2"><h3>Additional Information & Solution Suggestion</h3>'; 
     359            $errormessage  .= self::$exception_template.'</td></tr>'; 
     360        } 
     361 
     362        /** 
     363         * Panel 5 
     364         * 
     365         * Rapid Development 
     366         */ 
    361367        $placeholders = array(); 
    362368        # assign placeholders for replacements in the html 
    … …  
    382388        if(defined('DEVELOPMENT') and DEVELOPMENT == 1 and defined('RAPIDDEVTPL') and RAPIDDEVTPL == 1) 
    383389        { 
    384             $errormessage  .= '<tr><td colspan="2"><h3>Rapid Development</h3></td></tr>'; 
    385             $errormessage  .= '<tr><td colspan="2">'.self::getExceptionDevelopmentTemplate($placeholders).'</td></tr>'; 
    386  
    387             # Split 
    388             $errormessage  .= '<tr><td colspan="2">&nbsp;</td></tr>'; 
    389         } 
    390  
    391         # Backlink to Bugtracker with Exceptionmessage -> http://trac.clansuite.com/newticket 
     390            $errormessage  .= '<tr><td colspan="2"><h3>Rapid Development</h3>'; 
     391            $errormessage  .= self::getExceptionDevelopmentTemplate($placeholders).'</td></tr>'; 
     392        } 
     393 
     394        /** 
     395         * Panel 6 
     396         * 
     397         * Backlink to Bugtracker with Exceptionmessage 
     398         * @link http://trac.clansuite.com/newticket 
     399         */ 
    392400        $errormessage .= Clansuite_Errorhandler::getBugtrackerMessage($this->message, $this->file, $this->line, $this->trace); 
    393401 
    … …  
    395403        $errormessage   .= '</table>'; 
    396404 
    397         # Footer with Support-Backlinks 
     405        /** 
     406         * Panel 7 
     407         * 
     408         * Footer with Support-Backlinks 
     409         */ 
    398410        $errormessage  .= Clansuite_Errorhandler::getSupportBacklinks($this); 
    399411 
Note: See TracChangeset for help on using the changeset viewer.