Fix compatibility issues and add test

src/Url/WordPressGenerator.php

@@ -7,68 +7,42 @@
 class WordPressGenerator implements UrlGenerator
 {
     /**
-     * Determines if the generator supports the given package.
-     *
-     * @return bool
+     * {@inheritdoc}
      */
     public function supportsPackage(PackageInterface $package)
     {
-        return 0 === strpos($package->getName(), 'wpackagist-plugin/') || 0 === strpos($package->getName(), 'wpackagist-theme/');
+        return (bool) preg_match('#^wpackagist-(plugin|theme)/#', $package->getName());
     }
 
     /**
-     * Generates a compare URL for two versions of the same package.
-     *
-     * @return string|null
+     * {@inheritdoc}
      */
     public function getCompareUrl(PackageInterface $initialPackage, PackageInterface $targetPackage)
     {
         return null;
     }
 
     /**
-     * Generates URL for viewing a release or commit of a package.
-     *
-     * @return string|null
+     * {@inheritdoc}
      */
     public function getReleaseUrl(PackageInterface $package)
     {
         return null;
     }
 
     /**
-     * Generates URL for viewing the project page of a package (usually repository root).
-     *
-     * @return string|null
+     * {@inheritdoc}
      */
     public function getProjectUrl(PackageInterface $package)
     {
-        $type = $this->getPackageType($package);
+        preg_match('#wpackagist-(plugin|theme)/(.+)#', $package->getName(), $matches);
 
-        if (null === $type) {
+        if (empty($matches)) {
             return null;
         }
 
-        return sprintf('https://wordpress.org/%ss/%s', $type, $this->getPackageSlug($package));
-    }
-
-    /**
-     * @return string|null
-     */
-    protected function getPackageType(PackageInterface $package)
-    {
-        [$type] = explode('/', $package->getName(), 2);
-
-        return 0 === strpos($type, 'wpackagist-') ? substr($type, 11) : null;
-    }
-
-    /**
-     * @return string
-     */
-    protected function getPackageSlug(PackageInterface $package)
-    {
-        [, $slug] = explode('/', $package->getName(), 2);
+        list (, $type, $slug) = $matches;
 
-        return $slug;
+        return sprintf('https://wordpress.org/%ss/%s', $type, $slug);
     }
 }

tests/Url/WordPressGeneratorTest.php

@@ -6,6 +6,16 @@
 
 class WordPressGeneratorTest extends GeneratorTest
 {
+    public function testItSupportsOnlyWpackagistPackages()
+    {
+        $generator = $this->getGenerator();
+
+        $this->assertFalse($generator->supportsPackage($this->getPackage('acme/package', '3.12.1')));
+        $this->assertTrue($generator->supportsPackage($this->getPackage('wpackagist-plugin/my-plugin', '3.12.1')));
+        $this->assertTrue($generator->supportsPackage($this->getPackage('wpackagist-theme/my-theme', '3.12.1')));
+        $this->assertFalse($generator->supportsPackage($this->getPackage('acme-wpackagist-theme/my-theme', '3.12.1')));
+    }
+
     public function releaseUrlProvider()
     {
         return array(