Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

Fixed Nette 2.4-RC compatibility#125

Open
Olicek wants to merge 2 commits intojanmarek:master from
Olicek:patch-1
Open

Fixed Nette 2.4-RC compatibility #125
Olicek wants to merge 2 commits intojanmarek:master from
Olicek:patch-1

Conversation

@Olicek
Copy link

@Olicek Olicek commented May 4, 2016

No description provided.

public function afterCompile(Nette\PhpGenerator\ClassType $class)
{
$meta = $class->properties['meta'];
$meta = $class->getProperties()['meta'];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be $class->getProperty('meta') for short ...

mkapal, hlavacz, and radmax reacted with thumbs up emoji
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, works for me too.

Copy link

@TomasPilar TomasPilar Jul 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use shorter version $meta = $class->getProperty('meta'); :)

Copy link

radmax commented Jul 1, 2016

@Olicek it is now Nette 2.4 stable incompatibility

Copy link
Contributor

bckp commented Jul 12, 2016

Duplicate with #126 (that have better name by my opinion)

Copy link

@fprochazka Please can you merge this? :)

petak23, reflexator, radmax, spisovatelprogramu, jan-stanek, honzagula, and fmasa reacted with thumbs up emoji

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /