PhpOffice\PhpPresentation\Writer\PowerPoint2007\PptCharts::writeTypeDoughnut()

Complexity

Conditions	17
Paths	36

Size

Total Lines	149
Code Lines	84

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	81
CRAP Score	17.0301

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function writeTypeDoughnut(XMLWriter $objWriter, Doughnut $subject, $includeSheet = false)
    {
        // c:pieChart
        $objWriter->startElement('c:doughnutChart');

        // c:varyColors
        $objWriter->startElement('c:varyColors');
        $objWriter->writeAttribute('val', '1');
        $objWriter->endElement();

        // Write series
        $seriesIndex = 0;
        foreach ($subject->getSeries() as $series) {
            // c:ser
            $objWriter->startElement('c:ser');

            // c:idx
            $objWriter->startElement('c:idx');
            $objWriter->writeAttribute('val', $seriesIndex);
            $objWriter->endElement();

            // c:order
            $objWriter->startElement('c:order');
            $objWriter->writeAttribute('val', $seriesIndex);
            $objWriter->endElement();

            // c:tx
            $objWriter->startElement('c:tx');
            $coords = ($includeSheet ? 'Sheet1!$' . \PHPExcel_Cell::stringFromColumnIndex(1 + $seriesIndex) . '$1' : '');
            $this->writeSingleValueOrReference($objWriter, $includeSheet, $series->getTitle(), $coords);
            $objWriter->endElement();

            // Fills for points?
            $dataPointFills = $series->getDataPointFills();
            foreach ($dataPointFills as $key => $value) {
                // c:dPt
                $objWriter->startElement('c:dPt');
                $this->writeElementWithValAttribute($objWriter, 'c:idx', $key);
                // c:dPt/c:spPr
                $objWriter->startElement('c:spPr');
                $this->writeFill($objWriter, $value);
                // c:dPt/##c:spPr
                $objWriter->endElement();
                // ##c:dPt
                $objWriter->endElement();
            }

            // Write X axis data
            $axisXData = array_keys($series->getValues());

            // c:cat
            $objWriter->startElement('c:cat');
            $this->writeMultipleValuesOrReference($objWriter, $includeSheet, $axisXData, 'Sheet1!$A$2:$A$' . (1 + count($axisXData)));
            $objWriter->endElement();

            // Write Y axis data
            $axisYData = array_values($series->getValues());

            // c:val
            $objWriter->startElement('c:val');
            $coords = ($includeSheet ? 'Sheet1!$' . \PHPExcel_Cell::stringFromColumnIndex($seriesIndex + 1) . '$2:$' . \PHPExcel_Cell::stringFromColumnIndex($seriesIndex + 1) . '$' . (1 + count($axisYData)) : '');
            $this->writeMultipleValuesOrReference($objWriter, $includeSheet, $axisYData, $coords);
            $objWriter->endElement();

            $objWriter->endElement();

            ++$seriesIndex;
        }

        // c:dLbls
        $objWriter->startElement('c:dLbls');

        $this->writeElementWithValAttribute($objWriter, 'c:showLegendKey', $series->hasShowLegendKey() ? '1' : '0');

The variable $series seems to be defined by a foreach iteration on line 1231. Are you sure the iterator is never empty, otherwise this variable is not defined?

        $this->writeElementWithValAttribute($objWriter, 'c:showVal', $series->hasShowValue() ? '1' : '0');
        $this->writeElementWithValAttribute($objWriter, 'c:showCatName', $series->hasShowCategoryName() ? '1' : '0');
        $this->writeElementWithValAttribute($objWriter, 'c:showSerName', $series->hasShowSeriesName() ? '1' : '0');
        $this->writeElementWithValAttribute($objWriter, 'c:showPercent', $series->hasShowPercentage() ? '1' : '0');
        $this->writeElementWithValAttribute($objWriter, 'c:showBubbleSize', '0');
        $this->writeElementWithValAttribute($objWriter, 'c:showLeaderLines', $series->hasShowLeaderLines() ? '1' : '0');

        if ($series->hasDlblNumFormat()) {
            //c:numFmt
            $objWriter->startElement('c:numFmt');
            $objWriter->writeAttribute('formatCode', $series->getDlblNumFormat());
            $objWriter->writeAttribute('sourceLinked', '0');
            $objWriter->endElement();
        }

        // c:dLbls\c:txPr
        $objWriter->startElement('c:txPr');
        $objWriter->writeElement('a:bodyPr', null);
        $objWriter->writeElement('a:lstStyle', null);

        // c:dLbls\c:txPr\a:p
        $objWriter->startElement('a:p');

        // c:dLbls\c:txPr\a:p\a:pPr
        $objWriter->startElement('a:pPr');

        // c:dLbls\c:txPr\a:p\a:pPr\a:defRPr
        $objWriter->startElement('a:defRPr');
        $objWriter->writeAttribute('b', ($series->getFont()->isBold() ? 'true' : 'false'));
        $objWriter->writeAttribute('i', ($series->getFont()->isItalic() ? 'true' : 'false'));
        $objWriter->writeAttribute('strike', ($series->getFont()->isStrikethrough() ? 'sngStrike' : 'noStrike'));
        $objWriter->writeAttribute('sz', ($series->getFont()->getSize() * 100));
        $objWriter->writeAttribute('u', $series->getFont()->getUnderline());
        $objWriter->writeAttributeIf($series->getFont()->isSuperScript(), 'baseline', '30000');
        $objWriter->writeAttributeIf($series->getFont()->isSubScript(), 'baseline', '-25000');

        // c:dLbls\c:txPr\a:p\a:pPr\a:defRPr\a:solidFill
        $objWriter->startElement('a:solidFill');
        $this->writeColor($objWriter, $series->getFont()->getColor());
        $objWriter->endElement();

        // c:dLbls\c:txPr\a:p\a:pPr\a:defRPr\a:latin
        $objWriter->startElement('a:latin');
        $objWriter->writeAttribute('typeface', $series->getFont()->getName());
        $objWriter->endElement();

        // c:dLbls\c:txPr\a:p\a:pPr\a:defRPr\
        $objWriter->endElement();
        // c:dLbls\c:txPr\a:p\a:pPr\
        $objWriter->endElement();

        // c:dLbls\c:txPr\a:p\a:endParaRPr
        $objWriter->startElement('a:endParaRPr');
        $objWriter->writeAttribute('lang', 'en-US');
        $objWriter->writeAttribute('dirty', '0');
        $objWriter->endElement();

        // c:dLbls\c:txPr\a:p\
        $objWriter->endElement();
        // c:dLbls\c:txPr\
        $objWriter->endElement();

        $separator = $series->getSeparator();
        if (!empty($separator) && $separator != PHP_EOL) {
            // c:dLbls\c:separator
            $objWriter->writeElement('c:separator', $separator);
        }

        // c:dLbls\
        $objWriter->endElement();

        $this->writeElementWithValAttribute($objWriter, 'c:firstSliceAng', '0');
        $this->writeElementWithValAttribute($objWriter, 'c:holeSize', $subject->getHoleSize());

        $objWriter->endElement();
    }