day 6: Secure the Job Page

[spitou]

Hello,

Secure the job page don't work. I can not find my mistake, i have read several times the tutorial.

My apps/frontend/config/routing.yml

Code: Select all

# default rules
job:
  class:   sfPropelRouteCollection
  options: { model: JobeetJob }
   
job_show_user:
  url:     /job/:company_slug/:location_slug/:id/:position_slug
  class:   sfPropelRoute
  options:
    model: JobeetJob
    type:  object
    method_for_criteria: doSelectActive
  param:   { module: job, action: show }
  requirements:
    id: \d+


homepage:
  url:   /
  param: { module: job, action: index }

default_index:
  url:   /:module
  param: { action: index }

default:
  url:   /:module/:action/*


and my lib/model/JobeetJobPeer.php

Code: Select all

<?php

class JobeetJobPeer extends BaseJobeetJobPeer
{
  static public function getActiveJobs(Criteria $criteria = null)
  {
    if (is_null($criteria))
    {
      $criteria = new Criteria();
    }
   
    $criteria->add(JobeetJobPeer::EXPIRES_AT, time(), Criteria::GREATER_THAN);
    $criteria->addDescendingOrderByColumn(self::EXPIRES_AT);
   
    return self::doSelect($criteria);
  }

 
  static public function doSelectActive(Criteria $criteria)
  {
    $criteria->add(JobeetJobPeer::EXPIRES_AT, time(), Criteria::GREATER_THAN);
   
    return self::doSelectOne($criteria);
  }

}


if i try to get an expired job, the job is diplayed. I will not forward to a 404 page.

thank you in advance
lionel

[spitou]

URL with date expired:
http://jobeet.localhost/frontend_dev.php/job/sensio-labs/par is-france/42/web-developer-expired

So, the route "job_show_user" would be used.
Confirm with the debug bar "sfPatternRouting":
Match route "job_show_user" (/job/:company_slug/:location_slug/:id/:position_slug) for /job/sensio-labs/paris-france/42/web-developer-expired with parameters array ( 'module' => 'job', 'action' => 'show', 'company_slug' => 'sensio-labs', 'location_slug' => 'paris-france', 'id' => '42', 'position_slug' => 'web-developer-expired',)

But i think that i have a problem with method_for_criteria.

If i replace method_for_criteria: doSelectActive by
method_for_criteria: foo (foo don't exist). I have not an error on method_for_criteria not exist.

How can i know if the code pass in doSelectActive (lib/model/JobeetJobPeer.php) and the value of criteria?

[Progenitura]

Have you found a solution on this issue ? Because it's giving me headaches ~ ~ .

[erikhermeler]

i'm having the same problem, any help would be great ~ ~

[Ravi]

If method_for_criteria seems to doesn't work - change frontend/modules/job/actions/actions.class.php
method: executeShow(...)
to:

public function executeShow(sfWebRequest $request)
{
$this->article = $this->getRoute()->getObject();
}

[nervus70]

If method_for_criteria seems to doesn't work - change frontend/modules/job/actions/actions.class.php
method: executeShow(...)
to:

public function executeShow(sfWebRequest $request)
{
$this->article = $this->getRoute()->getObject();
}

it work thank you

[medresses]

It was a series of failures over time. Redudancy keeps the building up until a critical number of columns failed.