fix summaries formatter

Power-Components / livewire-powergrid

⚡ PowerGrid generates modern, powerful and easy-to-customize data tables using Laravel Livewire.

https://livewire-powergrid.com

MIT License

1.42k stars 209 forks source link

fix summaries formatter #1423

Closed emissarman closed 5 months ago

emissarman commented 5 months ago

⚡ PowerGrid - Pull Request

Welcome and thank you for your interest in contributing to our project!. You must use this template to submit a Pull Request or it will not be accepted.

Motivation

[x] Bug fix
[ ] Enhancement
[ ] New feature
[ ] Breaking change

Description

This Pull Request adds...

Related Issue(s): #_____.

Documentation

This PR requires Documentation update?

[ ] Yes
[x] No
[ ] I have already submitted a Documentation pull request.

luanfreitasdev commented 5 months ago

Hello, can you give a brief description of what PR is doing? Thanks for contributing

emissarman commented 5 months ago

@luanfreitasdev Hello, "Summarizing formatted data" did not work correctly in a situation where, for example, there are 10 cells in a column, and formatting was applied only to 3 columns and only for the sum and avg. Formatting rules for one column could be applied to another, which is incorrect behavior. I fixed this.