Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Environment Inheritance error #2

Closed
richlove1 opened this issue Apr 29, 2013 · 6 comments
Closed

Environment Inheritance error #2

richlove1 opened this issue Apr 29, 2013 · 6 comments

Comments

@richlove1
Copy link

Hi there

Latest update from Taylor includes and extra parameter in Environment.php Make() so now getting the following error:

Runtime Notice: Declaration of BigElephant\Presenter\View\Environment::make() should be compatible with Illuminate\View\Environment::make($view, $data = Array, $mergeData = Array) in /Web/tgc/vendor/bigelephant/presenter/src/BigElephant/Presenter/View/Environment.php line 9

(Still using BigElephant)

Rich

@richlove1
Copy link
Author

See: laravel/framework@0192d49

@richlove1
Copy link
Author

Possible fix:

     * Get a evaluated view contents for the given view.
     *
     * @param  string  $view
     * @param  array   $data
     * @param  array   $mergeData
     * @return Illuminate\View\View
     */
    public function make($view, $data = array(), $mergeData = array())
    {

        $data = array_merge($data, $mergeData);

        $data = $this->makePresentable($data);

        return parent::make($view, $data);
    }

@kasparasg
Copy link

This does the trick, hopefully this will get added to the package soon :)

@robclancy
Copy link
Owner

Okay on it.

@robclancy
Copy link
Owner

Can someone confirm this is fixed? I haven't got time to test other than phpunit

@richlove1
Copy link
Author

I've not tested extensively but it eradicates the error and follows logic of parent function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants