Needs review
Project:
mbase
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Sep 2016 at 12:44 UTC
Updated:
25 Sep 2016 at 15:52 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
arne_hortell commentedIt also requires to check IF ['class'] exists...
Comment #3
pifagorimplemented in patch
Comment #4
pifagorComment #5
arne_hortell commented// with no check if 'class' exists it gives notice message...
// creates typecasting if $link['attributes']['class'] is defined but is a string
if (array_key_exists('class',$link['attributes']))
$link['attributes']['class'] = array($link['attributes']['class'],'btn');
else
$link['attributes'][] = array('class'=>'btn');
Comment #6
arne_hortell commentedMore fixinig needed for mbase
Row 78 and forward should be
function _mbase_smart_region(&$var, $regions, $theme) {
foreach ($regions as $region) {
$drupal_block=false; // added by me
if (array_key_exists($region,$var['page'])) // added by me
$drupal_block = $var['page'][$region];
// Display the block if it exists.
Comment #7
pifagor@arne_hortell thank you for the advice. New patch
Comment #8
pifagorFixed on coding standards