Laravel: Get access to class-variable from public static function (basic oop issues)

Question

EDIT

In the meantime this question is been visited a few times. Just to share what I´ve learned with the help of stackoverflow and other resources I´d not recommend using the technique I was asking for. A cleaner way is to attach a variable containing the database-text within the controller:

$txt = Model::find(1);
return view('name', array('text' => $txt->content);

Now you can access text in your view like so

{{ $text ?? 'Default' }}

But if you´re currently also hustling with basic oop and/or mvc architecture read on. Maybe it helps :-)

ORIGINAL QUESTION

I´m trying to output some text loaded from db. This is my setup:

View:

{{ ContentController::getContent('starttext') }}

Controller:

class ContentController extends BaseController {

    public $text = '';

    public static function getContent($description)
    {
        $content = Content::where('description', $description)->get();
        $this->text = $content[0]->text;

        return $this->text;
    }

}

I was trying various ways to declare a class variable and access it in my function but I always end up with:

Using $this when not in object context

tbh I think I lack some basic oop knowledge :-D

A view does not have explicit access to the controller class ($this does not exist in that context). It's a basic MVC principle, separation of logic (controller) from presentation (view). The docs pretty clearly show this example: View::make('greeting', array(...stuff here...)) where "stuff" is whatever you need to pass to the view, from the controller. Let me put it this way: read the docs (laravel.com/docs/responses). — Sergiu Paraschiv
– Sergiu Paraschiv, Commented Jun 5, 2014 at 14:09
Also, you should first start with the docs on PHP OOP (php.net/manual/en/language.oop5.php). The code you wrote makes no sense whatsoever. — Sergiu Paraschiv
– Sergiu Paraschiv, Commented Jun 5, 2014 at 14:10
When sticking to MVC principles you´re definitely right. But why not using the mighty routing-system of Laravel for render some simple Views where no user input is given? — Kristo
– Kristo, Commented Jun 5, 2014 at 14:22
If it's Route::get '/' return View::make('homepage') then yes, don't over-engineer. It's not only about the user input. You have Eloquent DB access in a controller with a method that responds with a Blade view. Please don't do Eloquent queries in routes.php. Please do separate logic, presentation and data storage. — Sergiu Paraschiv
– Sergiu Paraschiv, Commented Jun 5, 2014 at 14:50

Populus · Accepted Answer · 2014-06-05 14:12:02Z

6

Static methods do not have access to $this. $this refers to an instantiated class (an object created with a new statement, e.g. $obj = new ContentController()), and static methods are not executed within an object.

What you need to do is change all the $this to self, e.g. self::$text to access a static variable defined in your class. Then you need to change public $text = ''; to public static $text = '';

This is why static methods/variables are bad practices most of the time...

Not an expert at Laravel, but I'm sure you don't need to use static methods to pass variables into templates... If that is the case, I'm staying the hell away from Laravel...

answered Jun 5, 2014 at 14:12

Populus

7,7003 gold badges41 silver badges56 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

The Alpha · Accepted Answer · 2014-06-06 01:21:58Z

You may try something like this (In the case of static):

class ContentController extends BaseController {

    public static $text = null;

    public static function getContent($description)
    {
        $content = Content::where('description', $description)->first();
        return static::$text = $content->text;
    }
}

Read the other answer to understand the difference; also read about Late Static Bindings but instead...

You may try something like this in Laravel to avoid static:

class ContentController extends BaseController {

    public $text = null;

    public function getContent($description)
    {
        $content = Content::where('description', $description)->first();
        return $this->text = $content->text;
    }
}

Use it like this:

{{ App::make('ContentController')->getContent('starttext') }}

Also this:

{{ with(new ContentController)->getContent('starttext') }}

Or this (Even without Laravel):

{{ (new ContentController)->getContent('starttext') }}

Chris L · Accepted Answer · 2018-12-18 14:41:34Z

1

In Laravel 5.7 the or operator has been removed so {{ $text or 'Default' }} doesn't work anymore. New operator is ??. Beginning with Laravel 5.7 it should be {{ $text ?? 'Default' }}

answered Dec 18, 2018 at 14:41

Chris L

11511 bronze badges

Collectives™ on Stack Overflow

Laravel: Get access to class-variable from public static function (basic oop issues)

Using $this when not in object context

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

Using $this when not in object context

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related