barryvdh / laravel-elfinder

elFinder bundle for Laravel
739 stars 171 forks source link

Config $options array has no effect. #68

Open clemblanco opened 9 years ago

clemblanco commented 9 years ago

Hi,

I can't manage to get anything customized using the $options array in the config file. 'resizable' => false doesn't do anything. However if I modify resources/views/ckeditor4.php by adding resize: false before the locale it works.

Thanks.

Stichoza commented 9 years ago

Same here.

iffifan commented 9 years ago

same here

kirant400 commented 9 years ago

options array are not merged. I found this code `function array_merge_recursive_distinct() { $arrays = func_get_args(); $base = array_shift($arrays);

if(!is_array($base)) $base = empty($base) ? array() : array($base);

foreach($arrays as $append) {
    if(!is_array($append)) $append = array($append);
    foreach($append as $key => $value) {
        if(!array_key_exists($key, $base) and !is_numeric($key)) {
            $base[$key] = $append[$key];
            continue;
        }
        if(is_array($value) or is_array($base[$key])) {
            $base[$key] = array_merge_recursive_distinct($base[$key], $append[$key]);
        }
        else if(is_numeric($key))
        {
            if(!in_array($value, $base)) $base[] = $value;
        }
        else {
            $base[$key] = $value;
        }
    }
}

return $base;

}`

kssee commented 9 years ago

Hi, I have a same issue here, can someone please fix it?! Thanks

foundedlabs commented 8 years ago

Hi guys is this still an issue?

It seems that https://github.com/barryvdh/laravel-elfinder/pull/92 is fixing this issue but the pull request has not been merged since last year...

Any idea of how to move this forward?

barryvdh commented 8 years ago

Wasn't this fixed with https://github.com/barryvdh/laravel-elfinder/commit/e8e307a950c73d01b90f0120ef5bf20228669dbc