aya-abdallah-FOO opened a new pull request, #6207:
URL: https://github.com/apache/fineract/pull/6207

    Fixes FINERACT-2725.                                                        
                                                                                
                      
                                                                                
                                                                                
                       
     `PaginationParameters.limitSql()` generates lowercase SQL clauses 
(`limit`/`offset`).                                                             
                                
     `DatabaseSpecificSQLGenerator.countQueryResult` was stripping these using  
                                                                                
                       
     case-sensitive regex, so on PostgreSQL the LIMIT clause survived into the  
                                                                                
                       
     `SELECT COUNT(*) FROM (...) AS temp` wrapper — causing paginated endpoints 
                                                                                
                       
     to return a `totalFilteredRecords` value capped at the page size instead 
of                                                                              
                         
     the true total.                                                            
                                                                                
                       
                                                                                
                                                                                
                       
     Made both patterns case-insensitive (`(?i)`) and use `\s+` for whitespace. 
  


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to