search

victorjonsson / Arlima

Article List Manager - Wordpress plugin suitable for online newspapers that's in need of a fully customizable front page
28 stars 16 forks source link

Check for path as array #19

Closed ekandreas closed 10 years ago

ekandreas commented 10 years ago

This issue regards classes/TemplatePathResolver.php line 56

Sometimes $path is an array. Perhaps check for it and convert the array to string? This causes a lot of error_log -messages.

if( is_array( $path ) && isset( $path[0] ) ) $path = $path[0]; $path = rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;

ekandreas commented 10 years ago

Should I create a fork and request a change instead?

chredd commented 10 years ago

Fork us and send a pull request. That would be awesome!

victorjonsson commented 10 years ago

Hi Andreas!

I have never seen this error on any of our websites. I'm suspecting that this error originates from you own code, when hooking into the filter arlima_template_paths. Could this be possible?

edit: The filter should return an one-dimensional array with paths to directories where your templates are located.

ekandreas commented 10 years ago

Victor! That is the case. Please ignore my request!