search

CodeSleeve / asset-pipeline

This Laravel 4 package provides a very simple and easy to use asset pipeline. It was heavily inspired by the Rails asset pipeline. We make use of the wonderful Assetic package to help with pre-compliation!
http://www.codesleeve.com
MIT License
489 stars 53 forks source link

error after updating to dev-master #146

Closed p1mps closed 10 years ago

p1mps commented 10 years ago

Hi all,

I have a problem: after updating to the last release I have this exception:

PHP Fatal error: Cannot access private property Codesleeve\AssetPipeline\AssetPipeline::$parser in /home/andrea/manager_new/vendor/codesleeve/asset-pipeline/src/Codesleeve/AssetPipeline/Commands/AssetsCleanCommand.php on line 74 {"error":{"type":"Symfony\Component\Debug\Exception\FatalErrorException","message":"Cannot access private property Codesleeve\AssetPipeline\AssetPipeline::$parser","file":"\/home\/andrea\/manager_new\/vendor\/codesleeve\/asset-pipeline\/src\/Codesleeve\/AssetPipeline\/Commands\/AssetsCleanCommand.php","line":74}}{"error":{"type":"Symfony\Component\Debug\Exception\FatalErrorException","message":"Cannot access private property Codesleeve\AssetPipeline\AssetPipeline::$parser","file":"\/home\/andrea\/manager_new\/vendor\/codesleeve\/asset-pipeline\/src\/Codesleeve\/AssetPipeline\/Commands\/AssetsCleanCommand.php","line":74}}

Any hints?

Thanks in advance

p1mps commented 10 years ago

Changing AssetPipeline's $parser property field form private to public fixed the issue.

Fractaliste commented 10 years ago

New getters should be used : #147

kdocki commented 10 years ago

@Fractaliste Thanks for the pull request. :+1: This fix has been merged into dev-master.

p1mps commented 10 years ago

Thanks to all :)

On Wed, Mar 12, 2014 at 6:18 PM, kdocki notifications@github.com wrote:

@Fractaliste https://github.com/Fractaliste Thanks for the pull request. [image: :+1:] This fix has been merged into dev-master.

Reply to this email directly or view it on GitHubhttps://github.com/CodeSleeve/asset-pipeline/issues/146#issuecomment-37436736 .

Andrea Imparato