-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
SyliusReportBundle #2422
SyliusReportBundle #2422
Conversation
Guys, looks interesting, but seriously you should squash those commits few tiems, over 90 is way to much to easily review ;) |
@@ -0,0 +1,41 @@ | |||
<?php | |||
|
|||
namespace Sylius\Bundle\CoreBundle\DataFetcher; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing license.
nice bundle |
$dateInterval = new \DateInterval($configuration['interval']); | ||
$numberOfPeriods = $configuration['start']->diff($configuration['end']); | ||
|
||
for ($i = 0; $i <= $numberOfPeriods->format($configuration['periodFormat']); $i++) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should avoid function call in loop, assign it to variable:
$numberOfPeriods = $configuration['start']->diff($configuration['end'])->format($configuration['periodFormat']);
for ($i = 0; $i <= $numberOfPeriods; $i++) {
}
79a1a66
to
82a4937
Compare
/** | ||
* @param Request $request | ||
* | ||
* @return Response |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be Symfony\Component\HttpFoundation\Response
- add use statement
@Zales0123 You use PHP 5.4 only syntax (array access) in TableRenderer and spec fails on 5.3. (see travis) Please remove these and it will turn green. |
b912870
to
b93c496
Compare
Just a bit of visual feedback - do the Time Period dropdown and the Filter button really need to be that loooooooooooooong? |
private $emails = array( | ||
'order_comment', | ||
'order_confirmation', | ||
'customer_welcome', | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove, wrong rebase.
Thanks guys, nice work! 👍 |
👍 |
SyliusReportBundle
SyliusReportBundle - flexible reports system, based on data fetchers and renderers. Now it's extremely easy to create a report with whatever data you want and nice look. Some default data fetchers and renderers already implemented, but adding another ones is a piece of cake. Find out more: Sylius/Sylius-Docs#251
![syliusreportbundle1](https://cloud.githubusercontent.com/assets/6212718/6003526/c0906e58-aafc-11e4-90f3-cfe0ba852ce7.jpg)
![syliusreportbundle2](https://cloud.githubusercontent.com/assets/6212718/6003528/c321e160-aafc-11e4-9ea5-609fed9a9159.jpg)
![syliusreportbundle3](https://cloud.githubusercontent.com/assets/6212718/6003530/c5bcfefa-aafc-11e4-9296-8aa90f980042.jpg)
![syliusreportbundle4](https://cloud.githubusercontent.com/assets/6212718/6003531/c8779768-aafc-11e4-925a-c301d687ff07.jpg)