Как исправить ошибку fatal error?

Question

[Александр]

Пытаюсь включить Jotcache на сайте на Joomla. Столкнулся с тем, что когда пытаюсь в настройках JotCache исключить модуль и позицию корзины из кэширования сайт просто падает и выводит в еррор.лог следующую ошибку.
PHP Fatal error: Cannot redeclare getEnding() (previously declared in C:\Windows\Temp\htmD867.tmp:10) in C:\Windows\Temp\htmD904.tmp on line 28

Код модуля привожу ниже. Где ошибка и как пофиксить это?

<?php
  $session = JFactory::getSession();
  $Goods = $session->get('Goods');
  $endingArray = array("услуга", "услуги", "услуг");
	if (count($Goods)!=0){
		echo ('<a href="/basket" class="basket">В корзине <span>'.count($Goods).'</span> <font>'.getEnding(count($Goods), $endingArray).'</font></a>');	
	}else{
		echo ('<a href="/basket" class="basket none">В корзине <span>0</span> <font>'.'</font></a>');	
}
	function getEnding($number, $endingArray)
	{
		$number = $number % 100;
		if ($number>=11 && $number<=19) {
			$ending=$endingArray[2];
		}
		else {
			$i = $number % 10;
			switch ($i)
			{
				case (1): $ending = $endingArray[0]; break;
				case (2):
				case (3):
				case (4): $ending = $endingArray[1]; break;
				default: $ending=$endingArray[2];
			}
		}
		return $ending;
	}
?>

Answer 1

[Алексей]

Вы, видимо, второй раз определяете функцию getEnding.
Как быстрое решение - замените в этом скрипте getEnding на getEnding1 в определении функции и ее вызове.

Comments

[Александр]

я уже так делал, и получил ту же самую ошибку только с именем getEnding1

[Алексей]

Понятно, видимо этот код включается дважды.
Надо выносить функцию куда-то "выше", чтобы она два раза не включалась в файл.

[Алексей]

Если не получится, то как быстрое решение (обратите внимание на знак $ при вызове функции):

<?php

  $getEnding =  function ($number, $endingArray)
  {
    $number = $number % 100;
    if ($number>=11 && $number<=19) {
      $ending=$endingArray[2];
    }
    else {
      $i = $number % 10;
      switch ($i)
      {
        case (1): $ending = $endingArray[0]; break;
        case (2):
        case (3):
        case (4): $ending = $endingArray[1]; break;
        default: $ending=$endingArray[2];
      }
    }
    return $ending;
  }

  $session = JFactory::getSession();
  $Goods = $session->get('Goods');
  $endingArray = array("услуга", "услуги", "услуг");
  if (count($Goods)!=0){
    echo ('<a href="/basket" class="basket">В корзине <span>'.count($Goods).'</span> <font>'.$getEnding(count($Goods), $endingArray).'</font></a>');	
  }else{
    echo ('<a href="/basket" class="basket none">В корзине <span>0</span> <font>'.'</font></a>');	
}
 
?>

[Александр]

Алексей: с вашим кодом сайт сразу упал, даже без кэша-) в еррор вывел [10-May-2016 14:39:35 Asia/Kuwait] PHP Parse error: syntax error, unexpected T_VARIABLE in C:\Windows\Temp\htm7698.tmp on line 23

[Александр]

лан, и так ясно что как-то модуль не так подключен видимо, буду разбираться

[Алексей]

Я просто забыл поставить ";" (правил код прямо тут).
Вот правильный код:

<?php

  $getEnding =  function ($number, $endingArray)
  {
    $number = $number % 100;
    if ($number>=11 && $number<=19) {
      $ending=$endingArray[2];
    }
    else {
      $i = $number % 10;
      switch ($i)
      {
        case (1): $ending = $endingArray[0]; break;
        case (2):
        case (3):
        case (4): $ending = $endingArray[1]; break;
        default: $ending=$endingArray[2];
      }
    }
    return $ending;
  };

  $session = JFactory::getSession();
  $Goods = $session->get('Goods');
  $endingArray = array("услуга", "услуги", "услуг");
  if (count($Goods)!=0){
    echo ('<a href="/basket" class="basket">В корзине <span>'.count($Goods).'</span> <font>'.$getEnding(count($Goods), $endingArray).'</font></a>');	
  }else{
    echo ('<a href="/basket" class="basket none">В корзине <span>0</span> <font>'.'</font></a>');	
}
 
?>

[Александр]

Алексей: воу, круто.
Спасибо