odiseoteam/syliusreportplugin

Report Plugin for Sylius. This plugin add a report interface to the Sylius administration. Some reports comes with this bundle but you can create your custom reports.

syliusreportplugin's Issues

Plugin inspiration

Hello Odiseo team!

My name is Mateusz, I work in Sylius company and one of my roles is Plugin Curator - I'm responsible for checking out Sylius Plugins that appear in our ecosystem.

I've encountered on this plugin some time ago - I'm really happy, that somebody wants to revive SyliusReportBundle as a Sylius plugin, as it was one of my first contributions to Sylius a few years ago. However, I think it would be nice to mention in plugin description that it's highly inspired on past Sylius bundle and component (and by highly inspired I mean it's almost all copy-pasted :)). I know we have not been supporting it for the last 2 years, I know it's released under MIT license. I just really believe, that open-source is mostly about mutual respect - and releasing someones work with all author blocks and commits history removed is a little violation of this rule :)

Best regards and have fun with Sylius plugins!

Mysql 'full_group_by' mode not compatible

For query

SELECT DATE(s0_.created_at) AS sclr_0, count(s0_.id) AS sclr_1, MONTH(s0_.created_at) AS sclr_2, YEAR(s0_.created_at) AS sclr_3 FROM sylius_order s0_ WHERE s0_.created_at >= ? AND s0_.created_at <= ? GROUP BY sclr_2, sclr_3 ORDER BY sclr_0 ASC, sclr_2 ASC, sclr_3 ASC' with params ["1919-09-03 00:00:00", "2019-09-03 23:59:59"

The error

Syntax error or access violation: 1055 Expression #1 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'gifts.s0_.created_at' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by").

[DOC REQUEST] How to create custom filter ?

Hey, the new release is great ! Thanks a lot for the good work.

The data fetcher are so easy to create but filter lack documentation tbh, could you please help a bit about that ? Thanks !

Class 'Sylius\Component\Order\Model\OrderInterface' does not exist

Hello,

Following the instructions I got this error when I choose to access a report

More informations :

This happen in dataFetcher, when they try to get metadata of OrderInterface class. The message mean that the class can't be found. Putting my own class in APpBundle\Entity work.

Exception during template rendering

Hi,

After my report creation, i got this excepetion :

Twig\Error\RuntimeError:
An exception has been thrown during the rendering of a template ("[Syntax Error] line 0, col 7: Error: Expected known function, got 'DATE'").

at vendor/odiseoteam/sylius-report-plugin/src/Resources/views/show.html.twig:64

Any information to debug that ?

Thanks a lot !

New version

Hello,

When do you plan to release the new version ?

Thank you !

Filter by category does not work for “Sales Total” reports.

According to the source code it needs a parent category named “category” which doesn’t fit our setup. I’m able to select all categories if I change findChildren(‘category’) to findRootNodes() in the QueryFilterFormBuilder. 
But even then it’s not optimal, as we have categories with the same name under different nodes. We need to see the tree here (maybe output the full path of a category like “Top level / level 1 / level 2” instead of just “level 2”).

PostgresSQL Support

I'm trying to add this plugin to our Sylius store, but we are using PostgresSQL as our database, I was able to use oroinc/doctrine-extensions to provide the SQL functions not available in beberlei/DoctrineExtensions for PostgresSQL.

I have run into an issue with queries using GROUP BY but that aren't including columns in the select list that aren't in the GROUP BY in an Aggregate function

An exception has been thrown during the rendering of a template ("An exception occurred while executing 'SELECT DATE(s0_.created_at) AS sclr_0, count(s0_.id) AS sclr_1, EXTRACT(MONTH FROM s0_.created_at) AS sclr_2, EXTRACT(YEAR FROM s0_.created_at) AS sclr_3 FROM sylius_order s0_ WHERE s0_.created_at >= ? AND s0_.created_at <= ? GROUP BY sclr_2, sclr_3 ORDER BY sclr_0 ASC, sclr_2 ASC, sclr_3 ASC' with params ["1920-04-02 00:00:00", "2020-04-02 16:08:04"]:

SQLSTATE[42803]: Grouping error: 7 ERROR: column "s0_.created_at" must appear in the GROUP BY clause or be used in an aggregate function
LINE 1: SELECT DATE(s0_.created_at) AS sclr_0, count(s0_.id) AS sclr...
^").

This is similar to issue #10 but that fix doesn't apply to Postgres.

Official Sylius plugin

Hello, @odiseoteam, one more time!

We would really love to display this plugin on our officially accepted plugins list (https://sylius.com/plugins/), as I strongly believe it could be useful for other people in the community :)

The only problem is, we have some strict requirements for the plugin that is meant to be displayed on our website. One of the most important is the properly developed test suite. Sadly, your plugin has neither unit (PHPSpec/PHPUnit) nor acceptance (Behat) tests. Do you think you would be able to provide them? It would be great if we could extend our accepted plugins set with SyliusReportPlugin :)

The same refers to your other plugins - none of them has tests, which is disqualifying for them to be official plugins.

If you have any question, please, feel free to ask me. You can also contact me at [email protected], if you want to talk about some ways we can help you with tests in your plugins :)

Nevertheless, thank you for choosing Sylius and for your great job!

odiseo_sylius_report.model.report.class param is missing

Hello,

Thank you for your work !

With a fresh install of this plugin with Sylius 1.4.3 and Symfony 4.2.7, there is this error : You have requested a non-existent parameter "odiseo_sylius_report.model.report.class".

Any idea how to fix that ? I could add a custom parameter but that's not really clean ...