Skip to content

Fix full circle pie slice (100%) drawing#5

Closed
grakic wants to merge 3 commits intoIcinga:masterfrom
grakic:patch-1
Closed

Fix full circle pie slice (100%) drawing#5
grakic wants to merge 3 commits intoIcinga:masterfrom
grakic:patch-1

Conversation

@grakic
Copy link
Copy Markdown
Contributor

@grakic grakic commented Aug 21, 2014

Having a single hostgroup, and all hosts and services up, landing page Pie Chart is empty. This patch set restores display of OK hosts and services. As a follow up, getPieSlicePath primitive can draw nice looking full circle pie slice.

piechart_circle

grakic added 3 commits August 21, 2014 03:50
Allows getPieSlicePath to draw a full circle pie slice (100%) by slightly adjusting endpoint to have a valid arc. Also hides a central line as there will be no more pie slices.
The getPieSlicePath primitive can draw it now.
Having a single hostgroup, and all hosts and services up, landing page Pie Chart is empty. This restores display of OK hosts and services.
@dnsmichi
Copy link
Copy Markdown
Contributor

Thanks for the patches! Could you please open a referencing issue at https://dev.icinga.org/projects/icingaweb2?jump=welcome so that the web developers may review it and schedule it for the next milestone. Thanks!

@lippserd lippserd self-assigned this Aug 21, 2014
@lippserd
Copy link
Copy Markdown
Member

@lippserd
Copy link
Copy Markdown
Member

Hi,

Sorry for the late response. Your commits are in the master branch already. I'll close this one.

Cheers,
Eric

@lippserd lippserd closed this Nov 14, 2014
DMajrekar added a commit to DMajrekar/icingaweb2 that referenced this pull request Nov 30, 2015
When users are in a different BaseDN to the groups, this causes no groups to be loaded out of Ldap. The following error is given:

2015-11-30T08:29:51+00:00 - ERROR - Stacktrace: #0 [internal function]: Icinga\Application\ApplicationBootstrap->Icinga\Application\{closure}(2, 'strpos(): Empty...', '/icingaweb2/lib...', 88, Array)
Icinga#1 /icingaweb2/library/Icinga/Repository/LdapRepository.php(88): strpos('<USER DN>', '')
Icinga#2 /icingaweb2/library/Icinga/Authentication/UserGroup/LdapUserGroupBackend.php(394): Icinga\Repository\LdapRepository->isRelatedDn('<USER DN>')
Icinga#3 /icingaweb2/library/Icinga/Authentication/UserGroup/LdapUserGroupBackend.php(487): Icinga\Authentication\UserGroup\LdapUserGroupBackend->isMemberAttributeAmbiguous()
Icinga#4 /icingaweb2/library/Icinga/Repository/Repository.php(436): Icinga\Authentication\UserGroup\LdapUserGroupBackend->initializeConversionRules()
Icinga#5 /icingaweb2/library/Icinga/Repository/Repository.php(590): Icinga\Repository\Repository->getConversionRules()
Icinga#6 /icingaweb2/library/Icinga/Repository/RepositoryQuery.php(329): Icinga\Repository\Repository->providesValueConversion('group', 'group_name')
Icinga#7 /icingaweb2/library/Icinga/Web/Widget/SortBox.php(126): Icinga\Repository\RepositoryQuery->order('group_name', 'asc')
Icinga#8 /icingaweb2/library/Icinga/Web/Controller.php(117): Icinga\Web\Widget\SortBox->handleRequest(Object(Icinga\Web\Request))
Icinga#9 /icingaweb2/application/controllers/GroupController.php(74): Icinga\Web\Controller->setupSortControl(Array, Object(Icinga\Repository\RepositoryQuery))
Icinga#10 /icingaweb2/library/vendor/Zend/Controller/Action.php(507): Icinga\Controllers\GroupController->listAction()
Icinga#11 /icingaweb2/library/Icinga/Web/Controller/Dispatcher.php(76): Zend_Controller_Action->dispatch('listAction')
Icinga#12 /icingaweb2/library/vendor/Zend/Controller/Front.php(937): Icinga\Web\Controller\Dispatcher->dispatch(Object(Icinga\Web\Request), Object(Icinga\Web\Response))
Icinga#13 /icingaweb2/library/Icinga/Application/Web.php(361): Zend_Controller_Front->dispatch(Object(Icinga\Web\Request), Object(Icinga\Web\Response))
Icinga#14 /icingaweb2/library/Icinga/Application/webrouter.php(109): Icinga\Application\Web->dispatch()
Icinga#15 /icingaweb2/public/index.php(4): require_once('/icingaweb2/lib...')
Icinga#16 {main}
@Crossovo Crossovo mentioned this pull request Nov 3, 2017
@MacFan4000 MacFan4000 mentioned this pull request Dec 11, 2019
@edsonblwolf edsonblwolf mentioned this pull request Jul 7, 2022
@whizps whizps mentioned this pull request Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants