Dynamic\Foxy\Controller\FoxyLinkGeneratorController::index()

Complexity

Conditions	7
Paths	7

Size

Total Lines	29
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function index()
    {
        $request = $this->getRequest();

        if (!$request->getVar('code')) {
            return 'Code is required';
        }

        if (!$request->getVar('price') && $request->getVar('price') != 0) {

It seems like you are loosely comparing $request->getVar('price') of type mixed|null to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

            return 'Price is required';
        }

        if (!$request->getVar('name')) {
            return 'Name is required';
        }

        $code = $request->getVar('code');
        $props = [];
        foreach ($request->getVars() as $var => $val) {
            $props[$var] = AddToCartForm::getGeneratedValue($code, $var, $val, 'value');
        }

        $post_url = '';
        foreach ($props as $key => $value) {
            $post_url .= $key . '=' . $value . '&';
        }
        $post_url = rtrim($post_url, '&');

        echo FoxyHelper::StoreURL() . '/cart?' . $post_url;
    }