3294 #943

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

dogmatic69 commented Nov 7, 2012

allow using array for $element

@markstory markstory commented on an outdated diff Nov 8, 2012

lib/Cake/Utility/Folder.php
* @return string Combined path
* @link http://book.cakephp.org/2.0/en/core-utility-libraries/file-folder.html#Folder::addPathElement
*/
public static function addPathElement($path, $element) {
- return rtrim($path, DS) . DS . $element;
+ if (!is_array($element)) {
+ $element = array($element);
+ }
@markstory

markstory Nov 8, 2012

Owner

Couldn't you just do

$element = (array)$element;
Contributor

dogmatic69 commented Dec 4, 2012

I have rebased 2.3 and updated the code.

Contributor

dogmatic69 commented Dec 4, 2012

The fail is a single find in postgresql getting the wrong data, doubt that has anything to do with this commit.

lorenzo closed this Jan 28, 2013

Owner

lorenzo commented Jan 28, 2013

Please reopen targeting the 2.4 branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment