ソースを参照

Disable /packages/ page as it is very inefficient

Jordi Boggiano 7 年 前
コミット
8fe5a9aff1

+ 1 - 18
src/Packagist/WebBundle/Controller/PackageController.php

@@ -41,28 +41,11 @@ use Packagist\WebBundle\Package\Updater;
 class PackageController extends Controller
 {
     /**
-     * @Template("PackagistWebBundle:Package:browse.html.twig")
      * @Route("/packages/", name="allPackages")
-     * @Cache(smaxage=900)
      */
     public function allAction(Request $req)
     {
-        $filters = array(
-            'type' => $req->query->get('type'),
-            'tag' => $req->query->get('tag'),
-        );
-
-        $data = $filters;
-        $page = $req->query->get('page', 1);
-
-        $packages = $this->getDoctrine()
-            ->getRepository('PackagistWebBundle:Package')
-            ->getFilteredQueryBuilder($filters);
-
-        $data['packages'] = $this->setupPager($packages, $page);
-        $data['meta'] = $this->getPackagesMetadata($data['packages']);
-
-        return $data;
+        return new RedirectResponse($this->generateUrl('browse'), Response::HTTP_MOVED_PERMANENTLY);
     }
 
     /**

+ 0 - 16
src/Packagist/WebBundle/Resources/views/Package/browse.html.twig

@@ -1,16 +0,0 @@
-{% extends "PackagistWebBundle:Web:list.html.twig" %}
-
-{% set filters = [] %}
-
-{% if type %}{% set filters = filters|merge(['browse.of_type'|trans({ '%type%': type|join('browse.join_or'|trans)})]) %}{% endif %}
-{% if tag %}{% set filters = filters|merge(['browse.with_tag'|trans({ '%tag%': tag|join('browse.join_or'|trans)})]) %}{% endif %}
-
-{% block content_title %}
-    <h1>{{ 'browse.packages'|trans }} {{ filters|join('browse.join_and'|trans) }}</h1>
-{% endblock %}
-
-{# TODO:
-
-- Add browsing by tag, type, most required, ..
-
-#}