UltraGrid ColumnFilter Error Cancel The title field is required!

1. Roger Blanchard

   

   425 posts
   Registered:
   29 Jun 2018
   15 Mar

   Link to this post

   When we turn on SmartFilterActive = True and SmartFilterUseOperators = True we will get an error message shown below. This is only on one column of the grid and happens to be an Integer. I have looked at the "BuildQueryExpressionForField" and actually displayed the results of the filter value and it is indeed an Integer. However, if I add code to save the object value to an ABL integer and then call IntegerHolder it works just fine.
   
   Any ideas? We use filters on other grids with columns defined as an Integer and have no issues.  
   
   Routine BuildQueryExpressionForField Consultingwerk.Windows.Util.UltraGridFilterRowHelper sent called routine IntegerHolder Consultingwerk.IntegerHolder mismatched parameters. (2570)
   
   BuildQueryExpressionForField Consultingwerk.Windows.Util.UltraGridFilterRowHelper at line 398  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\Windows\Util\UltraGridFilterRowHelper.r)
   BuildQueryExpression Consultingwerk.Windows.Util.UltraGridFilterRowHelper at line 279  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\Windows\Util\UltraGridFilterRowHelper.r)
   CollectFilterValuesHandler Consultingwerk.SmartComponents.Implementation.SmartDataBrowser at line 4689  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartDataBrowser.r)
   CollectFilterValuesHandler Osprey.CustomClasses.SmartComponents.OspreyDataBrowser at line 359  (C:\ProgressWrk\OspreyVer128107\OspreyFramework\Osprey\CustomClasses\SmartComponents\OspreyDataBrowser.r)
   OnCollectFilterValues Consultingwerk.SmartComponents.Base.SmartDatasetAdapter at line 3441  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Base\SmartDatasetAdapter.r)
   GetNewFilterValues Consultingwerk.SmartComponents.Base.SmartDatasetAdapter at line 1365  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Base\SmartDatasetAdapter.r)
   RetrieveData Consultingwerk.SmartComponents.Implementation.SmartBusinessEntityAdapter at line 5193  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartBusinessEntityAdapter.r)
   RetrieveData Osprey.CustomClasses.SmartComponents.OspreyBusinessEntityAdapter at line 142  (C:\ProgressWrk\OspreyVer128107\OspreyFramework\Osprey\CustomClasses\SmartComponents\OspreyBusinessEntityAdapter.r)
   PerformFilterRow Consultingwerk.SmartComponents.Implementation.SmartDataBrowser at line 4079  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartDataBrowser.r)
   OnKeyDown Consultingwerk.SmartComponents.Implementation.SmartDataBrowser at line 3727  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartDataBrowser.r)
   SetupApp.p at line 194  (C:\ProgressWrk\OspreyVer128107\OspreyPOS_BackOffice\Client\Pos\SetupApp.r)
   C:\ProgressWrk\OspreyVer128107\OspreyPOS_BackOffice\Client\Pos\OspreyPOS.p at line 516  (C:\ProgressWrk\OspreyVer128107\OspreyPOS_BackOffice\Client\Pos\OspreyPOS.r)
   adecomm/_runcode.p at line 678  (adecomm/_runcode.r)
   C:\Progress\OpenEdge\oeide\eclipse\plugins\com.openedge.pdt.debug.core_12.8.5.00\abl\_debuglauncher.p at line 109  (C:\Progress\OpenEdge\oeide\eclipse\plugins\com.openedge.pdt.debug.core_12.8.5.00\abl\_debuglauncher.p)
   
   Shown by:
   ShowErrorMessage Consultingwerk.Util.ErrorHelper at line 3323  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\Util\ErrorHelper.r)
   ShowErrorMessage Consultingwerk.Util.ErrorHelper at line 3201  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\Util\ErrorHelper.r)
   RetrieveData Consultingwerk.SmartComponents.Implementation.SmartBusinessEntityAdapter at line 5475  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartBusinessEntityAdapter.r)
   RetrieveData Osprey.CustomClasses.SmartComponents.OspreyBusinessEntityAdapter at line 142  (C:\ProgressWrk\OspreyVer128107\OspreyFramework\Osprey\CustomClasses\SmartComponents\OspreyBusinessEntityAdapter.r)
   PerformFilterRow Consultingwerk.SmartComponents.Implementation.SmartDataBrowser at line 4079  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartDataBrowser.r)
   OnKeyDown Consultingwerk.SmartComponents.Implementation.SmartDataBrowser at line 3727  (C:\ProgressWrk\OspreyVer128107\SmartComponentLibrary\Consultingwerk\SmartComponents\Implementation\SmartDataBrowser.r)
   SetupApp.p at line 194  (C:\ProgressWrk\OspreyVer128107\OspreyPOS_BackOffice\Client\Pos\SetupApp.r)
   C:\ProgressWrk\OspreyVer128107\OspreyPOS_BackOffice\Client\Pos\OspreyPOS.p at line 516  (C:\ProgressWrk\OspreyVer128107\OspreyPOS_BackOffice\Client\Pos\OspreyPOS.r)
   adecomm/_runcode.p at line 678  (adecomm/_runcode.r)
   C:\Progress\OpenEdge\oeide\eclipse\plugins\com.openedge.pdt.debug.core_12.8.5.00\abl\_debuglauncher.p at line 109  (C:\Progress\OpenEdge\oeide\eclipse\plugins\com.openedge.pdt.debug.core_12.8.5.00\abl\_debuglauncher.p)
2. Mike Fechner

   

   352 posts
   Registered:
   14 Sep 2016
   16 Mar in reply to Roger Blanchard

   Link to this post

   You seem to be on 12.8.5? And what's the SmartComponent Library version? 
   
   The error seems to occur when passing the value of UNBOX to the dynamically invoked constructor of the IntegerHolder class - which generally works fine. And as this is called fro BuildQueryExpression where we check for <> ? by unboxing and assigning the value to an integer, I would expect that the value of the filter cell is not the problem.
   
   Anything special with that integer field? A special format/input mask? Does it depend on the value the user enters?  
3. Roger Blanchard

   

   425 posts
   Registered:
   29 Jun 2018
   16 Mar in reply to Mike Fechner

   Link to this post

   Yes we are on 12.8.5.
   The SCL version is 11.7 #82635@2024-11-03
   
   >> Anything special with that integer field? A special format/input mask? Does it depend on the value the user enters?  
   
   There is no special format or InputMask. That is what I originally thought yesterday as this works elsewhere with an integer. I looked at it a dozen times but will look again. 
4. Roger Blanchard

   

   425 posts
   Registered:
   29 Jun 2018
   17 Mar in reply to Mike Fechner

   Link to this post

   I confirmed no Format or MaskInput. I also confirmed this is working for other columns that are integers. 
   
   Not sure what is going on.
5. Mike Fechner

   

   352 posts
   Registered:
   14 Sep 2016
   17 Mar in reply to Roger Blanchard

   Link to this post

   We can probably work around the issue by using an intermediate integer variable there. I just requires a few changes as we need to handle all the various data-types. 
   
   I will log an SCL ticket and we'll look into this. But it may take a few days until we get to this.
6. Roger Blanchard

   

   425 posts
   Registered:
   29 Jun 2018
   18 Mar in reply to Mike Fechner

   Link to this post

   Thanks Mike.
7. Mike Fechner

   

   352 posts
   Registered:
   14 Sep 2016
   Answered

   22 Mar in reply to Roger Blanchard

   Link to this post

   I have implemented a workaround now. In BuildQueryExpression, we already have primitive ABL variables - so I'm creating the Holder there now instead of the mix of DYNAMIC-NEW and UNBOX which basically causes two nested dynamic type references.
   
   I would expect that this will solve your issue. The workaround will be in the next release (next week or the following). If you need it earlier, please create an support desk ticket and we'll attach it there. 
   
   https://consultingwerk.atlassian.net/browse/SCL-4759 
8. Roger Blanchard

   

   425 posts
   Registered:
   29 Jun 2018
   24 Mar in reply to Mike Fechner

   Link to this post

   Thanks Mike!! The next release is perfect.
9. Roger Blanchard

   

   425 posts
   Registered:
   29 Jun 2018
   31 Mar in reply to Mike Fechner

   Link to this post

   Hello Mike,
   
   Was this change in the Feb 27 release? I do not see it listed in the release notes. It is not a big deal if it is not, but I did not want to grab this release if it was not present. 
   
   Thank you. 
10. Mike Fechner

    

    352 posts
    Registered:
    14 Sep 2016
    31 Mar in reply to Roger Blanchard

    Link to this post

    That would be going back in time :-)
    
    I'll release this today or tomorrow.
11. Roger Blanchard

    

    425 posts
    Registered:
    29 Jun 2018
    31 Mar in reply to Mike Fechner

    Link to this post

    You are correct!! I was thinking March 27th...not enough coffee yet. :)
    
    Thanks for the quick response.  
    
12. Roger Blanchard

    

    425 posts
    Registered:
    29 Jun 2018
    08 Apr

    Link to this post

    Hello Mike, 
    
    I just grabbed the latest release and the change did solve my issue.
    
    Thank you. 
13. Mike Fechner

    

    352 posts
    Registered:
    14 Sep 2016
    08 Apr in reply to Roger Blanchard

    Link to this post

    Thanks for the update, Roger! It was an educated guess ... so glad it works.