File tree Expand file tree Collapse file tree 2 files changed +37
-38
lines changed
PHP/CodeCoverage/Report/HTML Expand file tree Collapse file tree 2 files changed +37
-38
lines changed Original file line number Diff line number Diff line change @@ -211,29 +211,44 @@ protected function getBreadcrumbs(PHP_CodeCoverage_Report_Node $node)
211211
212212 foreach ($ path as $ step ) {
213213 if ($ step !== $ node ) {
214- $ breadcrumbs .= sprintf (
215- ' <li><a href="%s.html">%s</a> <span class="divider">/</span></li> ' . "\n" ,
216- $ step ->getId (),
217- $ step ->getName ()
218- );
214+ $ breadcrumbs .= $ this ->getInactiveBreadcrumb ($ step );
219215 } else {
220- $ breadcrumbs .= sprintf (
221- ' <li class="active">%s</li> ' . "\n" ,
222- $ step -> getName ()
216+ $ breadcrumbs .= $ this -> getActiveBreadcrumb (
217+ $ step ,
218+ $ node instanceof PHP_CodeCoverage_Report_Node_Directory
223219 );
224-
225- if ($ node instanceof PHP_CodeCoverage_Report_Node_Directory) {
226- $ breadcrumbs .= sprintf (
227- ' <li>(<a href="%s.dashboard.html">Dashboard</a>)</li> ' . "\n" ,
228- $ step ->getId ()
229- );
230- }
231220 }
232221 }
233222
234223 return $ breadcrumbs ;
235224 }
236225
226+ protected function getActiveBreadcrumb (PHP_CodeCoverage_Report_Node $ node , $ isDirectory )
227+ {
228+ $ buffer = sprintf (
229+ ' <li class="active">%s</li> ' . "\n" ,
230+ $ node ->getName ()
231+ );
232+
233+ if ($ isDirectory ) {
234+ $ buffer .= sprintf (
235+ ' <li>(<a href="%s.dashboard.html">Dashboard</a>)</li> ' . "\n" ,
236+ $ node ->getId ()
237+ );
238+ }
239+
240+ return $ buffer ;
241+ }
242+
243+ protected function getInactiveBreadcrumb (PHP_CodeCoverage_Report_Node $ node )
244+ {
245+ return sprintf (
246+ ' <li><a href="%s.html">%s</a> <span class="divider">/</span></li> ' . "\n" ,
247+ $ node ->getId (),
248+ $ node ->getName ()
249+ );
250+ }
251+
237252 protected function getCoverageBar ($ percent )
238253 {
239254 $ level = $ this ->getColorLevel ($ percent );
Original file line number Diff line number Diff line change @@ -228,29 +228,13 @@ protected function topProjectRisks(array $classes, $max = 10)
228228 return $ buffer ;
229229 }
230230
231- protected function getBreadcrumbs (PHP_CodeCoverage_Report_Node $ node )
231+ protected function getActiveBreadcrumb (PHP_CodeCoverage_Report_Node $ node, $ isDirectory )
232232 {
233- $ breadcrumbs = '' ;
234-
235- $ path = $ node ->getPathAsArray ();
236-
237- foreach ($ path as $ step ) {
238- if ($ step !== $ node ) {
239- $ breadcrumbs .= sprintf (
240- ' <li><a href="%s.html">%s</a> <span class="divider">/</span></li> ' . "\n" ,
241- $ step ->getId (),
242- $ step ->getName ()
243- );
244- } else {
245- $ breadcrumbs .= sprintf (
246- ' <li><a href="%s.html">%s</a></li> ' . "\n" .
247- ' <li class="active">(Dashboard)</li> ' . "\n" ,
248- $ step ->getId (),
249- $ step ->getName ()
250- );
251- }
252- }
253-
254- return $ breadcrumbs ;
233+ return sprintf (
234+ ' <li><a href="%s.html">%s</a></li> ' . "\n" .
235+ ' <li class="active">(Dashboard)</li> ' . "\n" ,
236+ $ node ->getId (),
237+ $ node ->getName ()
238+ );
255239 }
256240}
You can’t perform that action at this time.
0 commit comments