Added dynamic page titles for hot/latest listings and profile pages

sanjeevan · Jul 3, 2010 · 9a020ef · 9a020ef
1 parent c0c1d88
commit 9a020ef
Show file tree

Hide file tree

Showing 4 changed files with 67 additions and 2 deletions.
diff --git a/apps/frontend/modules/article/actions/actions.class.php b/apps/frontend/modules/article/actions/actions.class.php
@@ -26,6 +26,8 @@ public function executeHot(sfWebRequest $request)
 
     if (in_array($this->flavour, Article::getFlavours())){
       $q->where('a.flavour = ?', $this->flavour);
+    } else {
+      $this->flavour = 'all';
     }
 
     $cache_hash = array(
@@ -44,6 +46,19 @@ public function executeHot(sfWebRequest $request)
     $this->pager->init();
 
     $this->insertCodeFlavouredResources();
+
+    // page title
+    $title = '';
+    if ($this->flavour != 'all'){
+      $title .= 'Hot ' . Article::getFlavourName($this->flavour);
+      $title .= ' - Page ' . $request->getParameter('page', 1);
+      $title .= ' - ' . sfConfig::get('app_name');
+    } else {
+      $title .= sfConfig::get('app_tagline');
+      $title .= ' - Page ' . $request->getParameter('page', 1);
+      $title .= ' - ' . sfConfig::get('app_name');
+    }
+    $this->getResponse()->setTitle($title);
   }
 
   /**
@@ -62,6 +77,8 @@ public function executeLatest(sfWebRequest $request)
 
     if (in_array($this->flavour, Article::getFlavours())){
       $q->where('a.flavour = ?', $this->flavour);
+    } else {
+      $this->flavour = 'all';
     }
 
     $cache_hash = array(
@@ -80,6 +97,19 @@ public function executeLatest(sfWebRequest $request)
     $this->pager->init();
 
     $this->insertCodeFlavouredResources();
+
+    // page title
+    $title = '';
+    if ($this->flavour != 'all'){
+      $title .= 'Latest ' . Article::getFlavourName($this->flavour);
+      $title .= ' - Page ' . $request->getParameter('page', 1);
+      $title .= ' - ' . sfConfig::get('app_name');
+    } else {
+      $title .= sfConfig::get('app_tagline');
+      $title .= ' - Page ' . $request->getParameter('page', 1);
+      $title .= ' - ' . sfConfig::get('app_name');
+    }
+    $this->getResponse()->setTitle($title);
   }
 
   /**

diff --git a/apps/frontend/modules/article/templates/_show_link.php b/apps/frontend/modules/article/templates/_show_link.php
@@ -24,7 +24,7 @@
 
 <div class="summary">
   <?php if (strlen($article->getSummary()) > 0): ?>
-    <?php echo $article->getSummary(); ?>
+    <?php echo myUtil::markdown($article->getSummary()); ?>
   <?php endif; ?>
 </div>
 

diff --git a/apps/frontend/modules/profile/actions/actions.class.php b/apps/frontend/modules/profile/actions/actions.class.php
@@ -45,6 +45,8 @@ public function executeIndex(sfWebRequest $request)
 
     if (in_array($this->flavour, Article::getFlavours())){
       $q2->andWhere('a.flavour = ?', $this->flavour);
+    } else {
+      $this->flavour = 'all';
     }
 
     $this->pager = new sfDoctrinePager('Article', sfConfig::get('app_things_perpage'));
@@ -65,6 +67,16 @@ public function executeIndex(sfWebRequest $request)
         }
       }
     }
+
+    // set page title
+    $fullname = $this->user->getFirstname() . ' ' . $this->user->getLastname();
+    $flavour_filter = Article::getFlavourName($this->flavour);
+    $title = $fullname;
+    if (isset($flavour_filter)){
+      $title .= ' - ' . $flavour_filter;
+    }
+    $title .= ' - ' . sfConfig::get('app_name');
+    $this->getResponse()->setTitle($title);
   }
 
   public function executeAvatar(sfWebRequest $request)

diff --git a/lib/model/doctrine/Article.class.php b/lib/model/doctrine/Article.class.php
@@ -11,11 +11,34 @@
  */
 class Article extends BaseArticle
 {
+  static $valid_flavours = array(
+    'link'      => 'Links',
+    'code'      => 'Code',
+    'snapshot'  => 'Snapshots',
+    'question'  => 'Questions' 
+  );
+
+  public static function getFlavourName($flavour)
+  {
+    if (isset(self::$valid_flavours[$flavour])){
+      return self::$valid_flavours[$flavour];
+    }
+  }
+
+  /**
+   * Get valid flavours
+   * 
+   */
   public static function getFlavours()
   {
-    return array('link', 'code', 'snapshot', 'question');
+    return array_keys(self::$valid_flavours);
   }
 
+  /**
+   * Invalidate cache, the actual invalidation of the cache is run in a background
+   * process
+   * 
+   */
   public function invalidateCache()
   {
     /*