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

TASK: Refactor absolute namespaces in Flow / Command classes #469

Merged

Conversation

daniellienert
Copy link
Member

Additionally refactor legacy array syntax.

@mention-bot
Copy link

@daniellienert, thanks for your PR! By analyzing the annotation information on this pull request, we identified @kdambekalns, @ComiR and @robertlemke to be potential reviewers

Additionally refactor legacy array syntax.
@daniellienert daniellienert force-pushed the task/refactor-namespaces-in-command-classes branch from dfaab03 to 3b0d188 Compare September 1, 2016 13:47
@albe
Copy link
Member

albe commented Sep 7, 2016

The one Resource import could be also named PersistentResource, but since it's only used in the @var doc comment I see this not so much as issue. Looks good :)

@@ -189,7 +191,7 @@ public function cleanCommand()
}
}
}
$this->outputLine('Removed %s resource object(s) from the database.', array(count($brokenResources)));
$this->outputLine('Removed %s resource object(s) from the database.', [count($brokenResources)]);
break;
Copy link
Member

Choose a reason for hiding this comment

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

Why remove all this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sorry Christian - what do you mean?

Copy link
Member

Choose a reason for hiding this comment

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

Uhm, I think it was a display error in the diff view, now iwth the comments it looks normal so nevermind. It looked like the rest of the cases was deleted as well....

@kitsunet kitsunet merged commit 46b1b4b into neos:3.0 Sep 8, 2016
@daniellienert daniellienert deleted the task/refactor-namespaces-in-command-classes branch January 3, 2018 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants