fidley / falv

FALV - Fast ALV Grid
https://abapblog.com
GNU General Public License v3.0
141 stars 52 forks source link

Method falv->soft_refresh( ) #62

Closed Rino53 closed 3 years ago

Rino53 commented 3 years ago

Sometimes it is necessary to do a refresh_table_display without i_soft_refresh. So this option is moved to input parameter.

fidley commented 3 years ago

Hi @Rino53, as method is called soft_refresh I would keep it in original manner. There are two options what we could do:

  1. Use the standard method refresh_table_display
  2. Create a new method, refresh_stable, where you will basically use signature and implementation for the changed soft_refresh, like it looks in this pull request.
Rino53 commented 3 years ago

As you wish. I just wonder, why any one uses i_soft_refresh = true because it kills any sort criteria..

fidley commented 3 years ago

To be honest - I do. I use it frequently when there are no sums or sorting. Sometimes I even forbid that to the users, but not that often. :)