Jump to content

[deprecated] RockGrid - powerful, performant and beautiful data listing tool


bernhard

Recommended Posts

  Reveal hidden contents

 

Link to comment
Share on other sites

  Reveal hidden contents

 

Link to comment
Share on other sites

  On 8/26/2018 at 5:23 AM, Beluga said:

I can't believe I wasted 15 hours on this due to a missing echo.

Expand  

Don't you use Tracy Debugger? You could have easily done a bd($page->rockgrid) or d($page->rockgrid) and should have seen and been able to debug it quickly.

fhoKV8T.png

PS: Even if you lost 15 hours - the module took many many more to develop so you are still saving loads of hours ? 

  • Like 1
Link to comment
Share on other sites

Another oddity: I get sometimes 3-4 times the same page listed. Here, instead of showing 1060 pages, I get 2162:

rockgrid-showing-page-multiple-times.thumb.PNG.32751b90db833d76475a514687055733.PNG

My setup:

// example for template "project"
document.addEventListener('RockGridItemBeforeInit', function(e) {
	if(e.target.id != 'RockGridItem_project') return;
	var grid = RockGrid.getGrid(e.target.id);

	var col = grid.getColDef('vertec');
	col.headerName = 'Vertec';

	var col = grid.getColDef('title');
	col.headerName = "Titel";

	var col = grid.getColDef('client_name');
	col.headerName = "Kunde";

  var col = grid.getColDef('year');
  col.headerName = "Erscheinungsjahr";

  var col = grid.getColDef('service');
  col.headerName = "Dienstleistung";

  var col = grid.getColDef('product');
  col.headerName = "Worum geht's?";

  var col = grid.getColDef('project_desc_short');
  col.headerName = "Referenz-Beschreibung";

	// set fixed column width
	var col = grid.getColDef('created');
	col.headerName = grid.js.created;
	col.width = 150;
	col.suppressSizeToFit = true;

	// add coldef plugins (shortcuts for dates, numbers, etc)
	grid.addColDefPlugins({
		created: {name: 'date', format: 'DD.MM.YYYY HH:mm:ss'},
	});
});

document.addEventListener('RockGridItemAfterInit', function(e) {
	if(e.target.id != 'RockGridItem_transactions') return;
	var col;
	var colDef;
	var grid = RockGrid.getGrid(e.target.id);

	// hide one column
	grid.columnApi().setColumnVisible('myhiddencolumn', false);
});
<?php namespace ProcessWire;

// optional: load external asset
$this->rg->assets->add($this->config->paths->siteModules . 'FieldtypeRockGrid/lib/moment.min.js');

// set data for this grid via rockfinder
$this->setData(new RockFinder("template=project, parent=1041, sort=-created, include=all", [
    'title',
    'client_name',
    'vertec',
    'year',
    'service',
    'product',
    'project_desc_short',
    'created',
]));

Looking at the debug infos in getSQL:

SELECT
    `rockfinder`.*
FROM
    /* original pw query */
    (SELECT pages.id 
    FROM `pages` 
    WHERE (pages.templates_id=44) 
    AND (pages.parent_id=1041) 
    GROUP BY pages.id 
    ORDER BY pages.created DESC ) as `pwfinder`

/* rockfinder */
LEFT JOIN (    SELECT
      `pages`.`id` AS `id`,
      `title`.`title` AS `title`,
      `client_name`.`client_name` AS `client_name`,
      `vertec`.`vertec` AS `vertec`,
      `year`.`year` AS `year`,
      `service`.`service` AS `service`,
      `product`.`product` AS `product`,
      `project_desc_short`.`project_desc_short` AS `project_desc_short`,
      `created`.`created` AS `created`
    FROM
      `pages`
    
    /* --- join title --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `title`.`data` AS `title`
    FROM `field_title` AS `title`) AS `title` ON `title`.`pageid` = `pages`.`id`
    /* --- end title --- */
    
    
    /* --- join client_name --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `client_name`.`data` AS `client_name`
    FROM `field_client_name` AS `client_name`) AS `client_name` ON `client_name`.`pageid` = `pages`.`id`
    /* --- end client_name --- */
    
    
    /* --- join vertec --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `vertec`.`data` AS `vertec`
    FROM `field_vertec` AS `vertec`) AS `vertec` ON `vertec`.`pageid` = `pages`.`id`
    /* --- end vertec --- */
    
    
    /* --- join year --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `year`.`data` AS `year`
    FROM `field_year` AS `year`) AS `year` ON `year`.`pageid` = `pages`.`id`
    /* --- end year --- */
    
    
    /* --- join service --- */
    LEFT JOIN (SELECT
      `service`.`pages_id` AS `pageid`,
     `service`.`data` AS `service`
    FROM `field_service` AS `service`
    GROUP BY `service`.`pages_id`, `service`.`data`) AS `service` ON `service`.`pageid` = `pages`.`id`
    /* --- end service --- */
    
    
    /* --- join product --- */
    LEFT JOIN (SELECT
      `product`.`pages_id` AS `pageid`,
     `product`.`data` AS `product`
    FROM `field_product` AS `product`
    GROUP BY `product`.`pages_id`, `product`.`data`) AS `product` ON `product`.`pageid` = `pages`.`id`
    /* --- end product --- */
    
    
    /* --- join project_desc_short --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `project_desc_short`.`data` AS `project_desc_short`
    FROM `field_project_desc_short` AS `project_desc_short`) AS `project_desc_short` ON `project_desc_short`.`pageid` = `pages`.`id`
    /* --- end project_desc_short --- */
    
    
    /* --- join created --- */
    LEFT JOIN (SELECT `id` AS `pageid`, `created` FROM `pages` AS `created`) AS `created` ON `created`.`pageid` = `pages`.`id`
    /* --- end created --- */
    
) AS `rockfinder` ON `pwfinder`.`id` = `rockfinder`.`id`
/* end rockfinder */

imho, I think this is wrong:

    /* --- join title --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `title`.`data` AS `title`
    FROM `field_title` AS `title`) AS `title` ON `title`.`pageid` = `pages`.`id`
    /* --- end title --- */

Well, or any other part of the whole SQL - do you assume every title is necessarily unique? The only unique thing is the page id. Did you run into a similar wrong output, or do I have to adjust my query? (selector)

Link to comment
Share on other sites

I should have just used my brain. Luckily I am not prone to alcoholism, so this public embarrassment will not lead to pathological results.

Anyway, let's pretend nothing happened last week and move on to the next topic. I noticed the ag-Grid demo has a cool filter, which allows us to use multiple strings. However, when used with RockGrid/PW, it breaks, if the field values include null ones. Long story short, I figured the most inexpensive change perf-wise would be telling PDO to treat nulls as strings. I did the change in RockFinder. Perhaps it would be unwise to include in upstream as some users would expect nulls to stay nulls.

The change in RockFinder.module.php:

public function getObjects($array = null) {
  $timer = $this->timer('getObjects');
  try {
    $this->database->setAttribute(\PDO::ATTR_ORACLE_NULLS, \PDO::NULL_TO_STRING);
    $results = $this->database->query($this->getSql());
    $objects = $results->fetchAll($array ? \PDO::FETCH_ASSOC : \PDO::FETCH_OBJ);
  }

In RockGrid code, there is also a possibility to use sql query as datasource, but I don't see it in the interface. Anyway, in case one would need to use it somehow:

InputfieldRockGrid.module.php:

// sql query as datasource
$this->database->setAttribute(\PDO::ATTR_ORACLE_NULLS, \PDO::NULL_TO_STRING);
$results = $this->database->query($sql);
return $results->fetchAll(\PDO::FETCH_OBJ);

Edit: later I noticed that dealing with the nulls in JS does not seem to affect perf even with a big data set, so I modified the filter example below to just do if (value === null) value = '';

Using the filter from ag-Grid demo in your .js:

document.addEventListener('RockGridItemBeforeInit', function(e) {
  if(e.target.id != 'RockGridItem_rockgrid') return;
  var grid = RockGrid.getGrid(e.target.id);

  var col = grid.getColDef('myfield');
  col.headerName = grid.js.myfield;
  col.filter = PersonFilter;
  col.floatingFilterComponent = PersonFloatingFilterComponent;

});

document.addEventListener('RockGridItemAfterInit', function(e) {
  if(e.target.id != 'RockGridItem_rockgrid') return;
  var col;
  var colDef;
  var grid = RockGrid.getGrid(e.target.id);  
});

function PersonFilter() {
}

PersonFilter.prototype.init = function (params) {
    this.valueGetter = params.valueGetter;
    this.filterText = null;
    this.params = params;    
    this.setupGui();
};

// not called by ag-Grid, just for us to help setup
PersonFilter.prototype.setupGui = function () {
    this.gui = document.createElement('div');
    this.gui.innerHTML =
        '<div style="padding: 4px;">' +
        '<div><input style="margin: 4px 0px 4px 0px;" type="text" id="filterText" placeholder="Multi-string search..."/></div>' +
        '</div>';

    var that = this;
    this.onFilterChanged = function () {
        that.extractFilterText();
        that.params.filterChangedCallback();        
    };

    this.eFilterText = this.gui.querySelector('#filterText');
    this.eFilterText.addEventListener("input", this.onFilterChanged);
};

PersonFilter.prototype.extractFilterText = function () {
    this.filterText = this.eFilterText.value;
};

PersonFilter.prototype.getGui = function () {
    return this.gui;
};

PersonFilter.prototype.doesFilterPass = function (params) {
    // make sure each word passes separately, ie search for firstname, lastname
    var passed = true;
    var valueGetter = this.valueGetter;
    
    this.filterText.toLowerCase().split(" ").forEach(function (filterWord) {
        var value = valueGetter(params);
        if (value === null) value = '';
        if (value.toString().toLowerCase().indexOf(filterWord) < 0) {
            passed = false;
        }
    });

    return passed;
};

PersonFilter.prototype.isFilterActive = function () {
    var isActive = this.filterText !== null && this.filterText !== undefined && this.filterText !== '';
    return isActive;
};

PersonFilter.prototype.getModelAsString = function (model) {
    return model ? model : '';
};

PersonFilter.prototype.getModel = function () {
    return this.eFilterText.value;
};

// lazy, the example doesn't use setModel()
PersonFilter.prototype.setModel = function (model) {
    this.eFilterText.value = model;
    this.extractFilterText();
};

PersonFilter.prototype.destroy = function () {
    this.eFilterText.removeEventListener("input", this.onFilterChanged);
};

function PersonFloatingFilterComponent() { }

PersonFloatingFilterComponent.prototype.init = function (params) {
    this.params = params;
    this.eGui = document.createElement('input');
    var eGui = this.eGui;
    this.changeEventListener = function () {
        params.onFloatingFilterChanged(eGui.value);
    };
    this.eGui.addEventListener('input', this.changeEventListener);
};

PersonFloatingFilterComponent.prototype.getGui = function () {
    return this.eGui;
};

PersonFloatingFilterComponent.prototype.onParentModelChanged = function (model) {
    // add in child, one for each flat
    if (model) {
        this.eGui.value = model;
    } else {
        this.eGui.value = '';
    }
};

PersonFloatingFilterComponent.prototype.destroy = function () {
    this.eGui.removeEventListener('input', this.changeEventListener);
};

 

  • Like 1
Link to comment
Share on other sites

  On 8/27/2018 at 10:38 AM, dragan said:

I set up a simple RockGrid example, but see strange things in the backend, see screen capture and JS console errors (ignore the first error). Any idea what could cause this?

Expand  

This happens when you have a grid that tries to show more rows, then adds scrollbars and therefore does not have enough space to show those rows and removes one. Then it tries the same thing again. You can set your grid to a fixed amount of rows. The number of rows in your case do not change. It's only the number of pages that changes (and that's correct of course, if you add/remove one row you'll end up with different page numbers).

 

  On 8/27/2018 at 10:59 AM, dragan said:

Another oddity: I get sometimes 3-4 times the same page listed. Here, instead of showing 1060 pages, I get 2162:

Expand  

This can happen if you have page fields or repeaters with multiple items. The RockFinder might return those values as joined results with each referenced value in a separate row. Analyze your SQL or your RockFinder and the result itself to see where this comes from. You can comment out some fields of the RockFinder then you see which field causes the number of results to increase to more than you'd expect.

You can either create a different SQL (see the RockFinder docs, or - sorry - look into the code). Another way to prevent such situations is to create a new hidden field that gets populated via saveReady hook and holds the information you need. Eg you create a field "linked_pages" that holds an array of linked pages, eg

[{id:123,title:"demo page1"},{id:456,title:"demo page2"}]

You can then use a cellRenderer to show this information as you want (eg with icons, as comma separated list, as number of linked pages etc.). It might sound complex but it totally makes sense when you build more advanced grids.

Having said that, that's how it works right now. It's not perfect for sure. If you have any suggestions for improvement let me know.

 

  On 8/27/2018 at 11:04 AM, Beluga said:

However, when used with RockGrid/PW, it breaks, if the field values include null ones.

Expand  

In that case just setup a valueGetter and make sure you return a proper value for all cells. This is how you make sure it is a number (for calculating sums/avg etc):

document.addEventListener('RockGridItemBeforeInit', function(e) {
  if(e.target.id != 'RockGridItem_rockgrid') return;
  var grid = RockGrid.getGrid(e.target.id);

  var col = grid.getColDef('yourcolumn');
  col.valueGetter = function(params) {
    if(typeof params.data == 'undefined') return;
    var val = params.data[colDef.field];
    return val*1; // make sure the value is a number and not a string
  }
});

 

  On 8/27/2018 at 12:01 PM, Beluga said:

Another topic: are you able to hide the id column? I tried with the methods in the readme, but I can't make it go away.

Expand  

Of course you can hide it: https://www.ag-grid.com/javascript-grid-column-definitions/

document.addEventListener('RockGridItemBeforeInit', function(e) {
  if(e.target.id != 'RockGridItem_rockgrid') return;
  var grid = RockGrid.getGrid(e.target.id);

  var col = grid.getColDef('id');
  col.hide = true;

  var col = grid.getColDef('title');
  col.headerName = 'MyDemoTitle';
});

aBKM24C.png

 

  • Like 1
  • Thanks 1
Link to comment
Share on other sites

Cool, thanks for the hiding tip!

Hmm, now that I tested it, changing null to empty in JS does not seem to affect perf in any way. I actually did a quick JS flame graph comparison in Firefox and the relevant call took the same amount of milliseconds either way. So without any PDO hackery, the rockgrid.js PersonFilter.prototype.doesFilterPass in my previous example can be modified like this:

    this.filterText.toLowerCase().split(" ").forEach(function (filterWord) {
        var value = valueGetter(params);
        if (value === null) value = '';
        if (value.toString().toLowerCase().indexOf(filterWord) < 0) {
            passed = false;
        }
    });

Btw. has anyone tried this: https://www.ag-grid.com/javascript-grid-row-height/#auto-row-height

I tested it, but the result was just huge CPU chewing for a while (for a large data set), but row height did not adapt for cells with overflowing content. So simply col.autoHeight = true;

  • Like 1
Link to comment
Share on other sites

  On 8/27/2018 at 8:23 PM, bernhard said:

This can happen if you have page fields or repeaters with multiple items. The RockFinder might return those values as joined results with each referenced value in a separate row. Analyze your SQL or your RockFinder and the result itself to see where this comes from. You can comment out some fields of the RockFinder then you see which field causes the number of results to increase to more than you'd expect. 

Expand  

Hi. I'm searching since a couple of days why I get this behavior. I follow your example for pages but it never display with separator (in my picture id=1284 or 1286 or 1290). I have no idea why, maybe an expert eye could spot something.

$finder = new \ProcessWire\RockFinder('template=activite', ['title']); //really simple example
$field =$finder->addField('discipline', ['title']);  
$field->separator = "|";
return $finder->getSQL();

 

image.thumb.png.e3aad3ba80889919d6b8c4d27ba82c9c.png

SELECT
    `rockfinder`.*
FROM
    /* original pw query */
    (SELECT pages.id 
    FROM `pages` 
    WHERE (pages.templates_id=58) 
    AND (pages.status<1024) 
    GROUP BY pages.id ) as `pwfinder`

/* rockfinder */
LEFT JOIN (    SELECT
      `pages`.`id` AS `id`,
      `title`.`title` AS `title`,
      `discipline`.`discipline` AS `discipline`,
      `discipline`.`title` AS `discipline:title`
    FROM
      `pages`
    
    /* --- join title --- */
    LEFT JOIN (SELECT
      `pages_id` AS `pageid`,
      `title`.`data` AS `title`
    FROM `field_title` AS `title`) AS `title` ON `title`.`pageid` = `pages`.`id`
    /* --- end title --- */
    
    
    /* --- join discipline --- */
    LEFT JOIN (SELECT
      `discipline`.`pages_id` AS `pageid`,
     `discipline`.`data` AS `discipline`,
      GROUP_CONCAT(`title`.`data` ORDER BY `discipline`.`sort` SEPARATOR ',') AS `title`
    FROM `field_discipline` AS `discipline`
    LEFT JOIN `field_title` AS `title` ON `title`.`pages_id` = `discipline`.`data`
    GROUP BY `discipline`.`pages_id`, `discipline`.`data`) AS `discipline` ON `discipline`.`pageid` = `pages`.`id`
    /* --- end discipline --- */
    
) AS `rockfinder` ON `pwfinder`.`id` = `rockfinder`.`id`
/* end rockfinder */

Thanks!!

  • Thanks 1
Link to comment
Share on other sites

  On 8/28/2018 at 11:36 AM, dragan said:

Well, over here, when I try to open a page for editing, it never gets loaded. It just keeps on loading.... forever.

Expand  

I'm quite sure that this has nothing to do with either RockGrid or RockFinder. Please try a clean install and report back if you have any problems. Thanks.

Link to comment
Share on other sites

Hi @jmartsch thanks for the hint.

Regarding the alpha state: Yes. Still alpha. Whatever that means. I'm using it on several sites in production but I'm still developing things and sometimes change some concepts, so there might be breaking changes in the future. That's why it is alpha and I don't plan to change that, but it's working great for all my scenarios and it's already a huge step forward compared to the old datatables module/library ? 

Link to comment
Share on other sites

  On 9/4/2018 at 4:19 PM, bernhard said:

@Beluga working on filters right now and just saw that your example could also be done with the quickfilter: https://www.ag-grid.com/javascript-grid-filter-quick/

maybe that's interesting for you?

Expand  

Now that I looked into it, it does not seem to do what I need. The example has a separate field that filters all the columns. I couldn't figure out how to make it work so it would act as a "normal" column filter. It seems simpler to just continue using the PersonFilter thing.

Link to comment
Share on other sites

Hey Bernhard, I get an error that moment.js could not be loaded.

Then I found a date.md file which says 

  Quote
You need to include `moment.js` to your assets:
```php
$this->rg->assets->add($this->config->paths->siteModules . 'FieldtypeRockGrid/lib/moment.min.js');
```
Expand  

short note: the name of the module in this file is still RockGrid instead of FieldtypeRockGrid.

So I tried to do this, but now I get an error `Call to a member function add() on null`

What am I doing wrong? Here is my code. I use RockGrid inside of a module:

 $f = $this->modules->get('InputfieldRockGrid');
    $f->rg->assets->add($this->config->paths->siteModules . 'FieldtypeRockGrid/lib/moment.min.js');
    $f->name = 'stats';
    $f->themeBorder = 'none';
    $f->height = 0;
    $f->pageSize = 50; // set initial pagination to 25 rows

    $finder = new RockFinder('template=stelle', ['title', 'created']);
    $field = $finder->addField('mitarbeiterId', ['vorname', 'nachname']);

// $sql = $finder->getSQL();
// $finder->sql = "SELECT id, SUM(value) AS revenue, DATE_FORMAT(date, '%Y-%m') AS dategroup FROM ($sql) AS tmp GROUP BY dategroup";
// d($finder->getObjects());

    $f->setData($finder);

    return $f->render();

 

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...