LaravelDaily / laravel-charts

Package to draw charts in Laravel with Chart.js
MIT License
531 stars 118 forks source link

Something went wrong #96

Open Zaibali9999 opened 1 year ago

Zaibali9999 commented 1 year ago

Screenshot (121) Sometimes dates are not showing and counting is also wrong

PovilasKorop commented 1 year ago

@Zaibali9999 could you help reproduce? What is your data, and what are the parameters of your chart?

Zaibali9999 commented 1 year ago
        $settings1 = [
            'chart_title'           => 'Blocked',
            'chart_type'            => 'line',
            'report_type'           => 'group_by_date',
            'model'                 => 'App\ Model ',
            'group_by_field'        => 'created_at',

            "group_by_period" => "day",
            "aggregate_function" => "sum",

            'conditions'            => [
                ['name' => 'status', 'condition' => 'status = "Blocked"',

'color' => 'red', 'fill' => false],

            ],

        ];
        $settings2 = [
            'chart_title'           => 'Bypassed',
            'chart_type'            => 'line',
            'report_type'           => 'group_by_date',
            'model'                 => 'App\Model',
            'group_by_field'        => 'created_at',

            "group_by_period" => "day",
            "aggregate_function" => "sum",

            'conditions'            => [
                ['name' => 'status', 'condition' => 'status = "Bypassed"

', 'color' => 'green', 'fill' => false],

            ],

        ];

        $chart = new LaravelChart($settings1,$settings2);

On Sun, Feb 26, 2023 at 5:34 PM PovilasKorop @.***> wrote:

@Zaibali9999 https://github.com/Zaibali9999 could you help reproduce? What is your data, and what are the parameters of your chart?

— Reply to this email directly, view it on GitHub https://github.com/LaravelDaily/laravel-charts/issues/96#issuecomment-1445376540, or unsubscribe https://github.com/notifications/unsubscribe-auth/APUOQXFXDQZMP6HXLUEJ7ETWZNSVTANCNFSM6AAAAAAVIPYCTY . You are receiving this because you were mentioned.Message ID: @.***>

PovilasKorop commented 1 year ago

@Zaibali9999 could you also prepare the migration/seed of the data, so we could reproduce it on our end?

Zaibali9999 commented 1 year ago

Actually I'm working on SQL and XSS payloads Storing data into database For SQL payloads it works fine For XSS count is not working properly

PovilasKorop commented 1 year ago

@Zaibali9999 well if I can't reproduce exactly the same situation, I can't help you with debugging it, sorry.

Zaibali9999 commented 1 year ago

Thank you so much for reply can you please share your email so i can share information there

On Wed, Mar 1, 2023 at 7:50 AM PovilasKorop @.***> wrote:

@Zaibali9999 https://github.com/Zaibali9999 well if I can't reproduce exactly the same situation, I can't help you with debugging it, sorry.

— Reply to this email directly, view it on GitHub https://github.com/LaravelDaily/laravel-charts/issues/96#issuecomment-1449339820, or unsubscribe https://github.com/notifications/unsubscribe-auth/APUOQXGNDQIXPNZTGYNOETLWZ3ISXANCNFSM6AAAAAAVIPYCTY . You are receiving this because you were mentioned.Message ID: @.***>

PovilasKorop commented 1 year ago

povilas@laraveldaily.com or you can invite me to github, username povilaskorop