Bruno17 / MIGX

MIGX for revo 2.2 and above
83 stars 78 forks source link

MultiTVGrids Setting Other Grids extended from LocalGrid to EnableRowBody False #330

Open freelancewebdev opened 5 years ago

freelancewebdev commented 5 years ago

As it says in the title, multiTV grids appear to be setting other grids extended from LocalGrid on the same manager page to enableRowBody:false.

I noticed this on a custom resource I'm building at the moment and replicated it on a regular resource using the resource groups grid which is extended from LocalGrid.

Viewing the source script produced by MIGX for its grids, it's not immediately apparent to me why it would be preventing the enableRowBody config to pass up the superclass chain but I wonder if it might have something to do with the grid being added to the MODx object grids object at the same level as LocalGrid while at the same time being extended from LocalGrid. Seems like there may be the possibility for multitv grid, which has enableRowBody not explicitly set and is therefore false, to be writing that config option back to the localGrid superclass after other grid classes like mine and the resource group grids but before these grids rendered.

https://forums.modx.com/thread/104772/custom-localgrid---enablerowbody-not-being-applied#dis-post-563455