Icinga / icingaweb2-module-cube

Drill-down view for Icinga web 2 based on custom variables
GNU General Public License v2.0
45 stars 12 forks source link

Fixing an error when mixing certain dimension together #28

Closed rudexi closed 5 years ago

rudexi commented 6 years ago

Upon using Cube for the first time, I cam upon an issue immediately. Whenever I was using two dimensions, the error:

Undefined property: stdClass::$mydimension

A quick search told me it was a standard php error:

https://stackoverflow.com/questions/8496855/undefined-property-stdclass

and it was resolved quickly. My commit fix the issue I was experimenting.

nbuchwitz commented 6 years ago

This fix only suppresses the output of the missing totals, which is caused by wrong rollup syntax for PostgreSQL databases. PR #29 should fix this.

nilmerg commented 5 years ago

Aye, that's just a work-around and #29 is now merged.