Почему этот Zend Framework _redirect () не работает? - PullRequest
0 голосов
/ 19 сентября 2009

Я занимаюсь разработкой приложения для Facebook в Zend Framework. В startAction () я получаю следующую ошибку:

URL http://apps.facebook.com/rails_across_europe/turn/move-trains-auto недействителен.

Я включил код для startAction () ниже. Я также включил код для moveTrainsAutoAction (это все действия TurnController). Я не могу найти ничего плохого в своей функции _redirect () в startAction (). Я использую тот же редирект в других действиях, и они выполняются безупречно. Пожалуйста, просмотрите мой код и дайте мне знать, если вы обнаружите проблему? Я ценю его! Спасибо.

  public function startAction() {
    require_once 'Train.php';
    $trainModel = new Train();

    $config = Zend_Registry::get('config');

    require_once 'Zend/Session/Namespace.php';
    $userNamespace = new Zend_Session_Namespace('User');
    $trainData = $trainModel->getTrain($userNamespace->gamePlayerId);

    switch($trainData['type']) {
      case 'STANDARD':
      default:
        $unitMovement = $config->train->standard->unit_movement;
        break;
      case 'FAST FREIGHT':
        $unitMovement = $config->train->fast_freight->unit_movement;
        break;
      case 'SUPER FREIGHT':
        $unitMovement = $config->train->superfreight->unit_movement;
        break;
      case 'HEAVY FREIGHT':
        $unitMovement = $config->train->heavy_freight->unit_movement;
        break;
    }
    $trainRow = array('track_units_remaining' => $unitMovement);
    $where = $trainModel->getAdapter()->quoteInto('id = ?', $trainData['id']);
    $trainModel->update($trainRow, $where);
    $this->_redirect($config->url->absolute->fb->canvas . '/turn/move-trains-auto');
  }
.
.
.
  public function moveTrainsAutoAction() {
$log = Zend_Registry::get('log');
$log->debug('moveTrainsAutoAction');
    require_once 'Train.php';
    $trainModel = new Train();

    $userNamespace = new Zend_Session_Namespace('User');
    $gameNamespace = new Zend_Session_Namespace('Game');

    $trainData = $trainModel->getTrain($userNamespace->gamePlayerId);

    $trainRow = $this->_helper->moveTrain($trainData['dest_city_id']);
    if(count($trainRow) > 0) {
      if($trainRow['status'] == 'ARRIVED') {
        // Pass id for last city user selected so we can return user to previous map scroll postion
        $this->_redirect($config->url->absolute->fb->canvas . '/turn/unload-cargo?city_id='.$gameNamespace->endTrackCity);
      } else if($trainRow['track_units_remaining'] > 0) {
        $this->_redirect($config->url->absolute->fb->canvas . '/turn/move-trains-auto');
      } else { /* Turn has ended */
        $this->_redirect($config->url->absolute->fb->canvas . '/turn/end');
      }
    }
    $this->_redirect($config->url->absolute->fb->canvas . '/turn/move-trains-auto-error'); //-set-destination-error');
  }

Ответы [ 3 ]

2 голосов
/ 19 сентября 2009

Как отмечает @ Jani Hartikainen в своем комментарии, на самом деле нет необходимости кодировать подчеркивания URL. Попробуйте перенаправить с помощью литеральных подчеркиваний и посмотрите, работает ли это, поскольку я считаю, что редирект делает свою собственную кодировку URL.


На самом деле не имеет отношения к вашему вопросу, но, по моему мнению, вам следует немного изменить свой код, чтобы избавиться от операторов switch-case (или, по крайней мере, локализовать их в одной точке):

Контроллеры / TrainController.php

[...]
public function startAction() {
    require_once 'Train.php';
    $trainTable = new DbTable_Train();

    $config = Zend_Registry::get('config');

    require_once 'Zend/Session/Namespace.php';
    $userNamespace = new Zend_Session_Namespace('User');
    $train = $trainTable->getTrain($userNamespace->gamePlayerId);

    // Add additional operations in your getTrain-method to create subclasses
    // for the train
    $trainTable->trackStart($train);
    $this->_redirect(
       $config->url->absolute->fb->canvas . '/turn/move-trains-auto'
    );
  }
  [...]

Модели / dbTable / Train.php

  class DbTable_Train extends Zend_Db_Table_Abstract
  {
     protected $_tableName = 'Train';
     [...]
     /**
      *
      *
      * @return Train|false The train of $playerId, or false if the player
      * does not yet have a train
      */
     public function getTrain($playerId)
     {
         // Fetch train row
         $row = [..];
         return $this->trainFromDbRow($row);

     }
     private function trainFromDbRow(Zend_Db_Table_Row $row)
     {
         $data = $row->toArray();
         $trainType = 'Train_Standard';
         switch($row->type) {
           case 'FAST FREIGHT':
             $trainType = 'Train_Freight_Fast';
             break;
           case 'SUPER FREIGHT':
             $trainType = 'Train_Freight_Super';
             break;
           case 'HEAVY FREIGHT':
             $trainType = 'Train_Freight_Heavy';
             break;
         }
         return new $trainType($data);
     }

     public function trackStart(Train $train)
     {
         // Since we have subclasses here, polymorphism will ensure that we 
         // get the correct speed etc without having to worry about the different
         // types of trains.
         $trainRow = array('track_units_remaining' => $train->getSpeed());
         $where = $trainModel->getAdapter()->quoteInto('id = ?', $train->getId());
         $this->update($trainRow, $where);
     }
     [...]

/ модель / Train.php

abstract class Train
{

   public function __construct(array $data)
   {
      $this->setValues($data);
   }

   /**
    * Sets multiple values on the model by calling the
    * corresponding setter instead of setting the fields
    * directly. This allows validation logic etc
    * to be contained in the setter-methods.
    */
   public function setValues(array $data)
   {
      foreach($data as $field => $value)
      {
         $methodName = 'set' . ucfirst($field);
         if(method_exists($methodName, $this))
         {
            $this->$methodName($value);
         }
      }
   }
   /**
    * Get the id of the train. The id uniquely
    * identifies the train.
    * @return int
    */
   public final function getId () 
   {
      return $this->id;
   }
   /**
    * @return int The speed of the train / turn
    */
   public abstract function getSpeed ();
   [..] //More common methods for trains
}

/ модели / Поезда / Standard.php

class Train_Standard extends Train
{
    public function getSpeed ()
    {
       return 3;
    }
    [...]
}

/ модели / Поезда / Грузовые / Super.php

class Train_Freight_Super extends Train
{
    public function getSpeed ()
    {
       return 1;
    }

    public function getCapacity ()
    {
       return A_VALUE_MUCH_LARGER_THAN_STANDARD;
    }
    [...]
}
0 голосов
/ 19 сентября 2009

Я думаю, что, возможно, нашел ответ. Похоже, что Facebook плохо работает с редиректом, поэтому необходимо использовать FBML «fb: redirect». Это похоже на работу:

$this->_helper->layout()->disableLayout();
$this->_helper->viewRenderer->setNoRender();

echo '<fb:redirect url="' . $config->url->absolute->fb->canvas . '/turn/move-trains-auto"/>';
0 голосов
/ 19 сентября 2009

По умолчанию отправляется HTTP 302 Redirect . Поскольку он записывает заголовки, если какой-либо вывод записывается в вывод HTTP, программа прекращает отправку заголовков. Попробуйте посмотреть на запросы и ответы внутри Firebug .

В противном случае попробуйте использовать параметры не по умолчанию для метода _redirect () . Например, вы можете попробовать:


$ropts = { 'exit' => true, 'prependBase' => false };
$this->_redirect($config->url->absolute->fb->canvas . '/turn/move-trains-auto', $ropts);

Существует еще одна интересная опция для метода _redirect () , опция code , которую можно отправить, например, HTTP 301 Moveed Permanently code.


$ropts = { 'exit' => true, 'prependBase' => false, 'code' => 301 };
$this->_redirect($config->url->absolute->fb->canvas . '/turn/move-trains-auto', $ropts);
...