Closed nilmerg closed 2 years ago
My basic host cube implementation can be found here. (Requires https://github.com/Icinga/ipl-sql/pull/48)
The main differences compared to #64 are:
CubeRenderer
Icinga\Module\Cube\CubeRenderer
DbCube
ZfSelectWrapper
Icinga\Module\Cube\Ido
customvar_flat
customvar
Things to change:
IcingaDbHostStatusCube
IcingaDbServiceStatusCube
IcingaDbCube
fetchAll()
Icinga\Module\Cube\IcingaDb\CustomVariableDimension::addToCube()
IcingaDbCube::fetchAll()
/** * Create additional action links for the given cube * * @param Icinga\Module\Cube\IcingaDb\IcingaDbCube $cube * * @return ipl\Web\Widget\Link[] */ createActionLinks(Icinga\Module\Cube\IcingaDb\IcingaDbCube $cube)
My basic host cube implementation can be found here. (Requires https://github.com/Icinga/ipl-sql/pull/48)
The main differences compared to #64 are:
CubeRenderer
classes have moved to their own namespace:Icinga\Module\Cube\CubeRenderer
DbCube
andZfSelectWrapper
have moved to the namespaceIcinga\Module\Cube\Ido
customvar_flat
instead ofcustomvar
Things to change:
IcingaDbHostStatusCube
to be final, generally. But classIcingaDbServiceStatusCube
isn't implemented yetIcingaDbCube
mostly consists of the methodfetchAll()
. It doesn't differentiate between inner, full and rollup queries, which it should IMHOIcinga\Module\Cube\IcingaDb\CustomVariableDimension::addToCube()
needs to be implemented. Note my todo's in methodIcingaDbCube::fetchAll()