support package look up for find command

Author: themsaid

README.md

@@ -6,7 +6,7 @@ translation lines with ease. Taking care of a multilingual interface is not a he
 <br>
 <br>
 
-<img src="http://s30.postimg.org/ni241hhpd/ezgif_com_resize.gif">
+<img src="http://s15.postimg.org/3lfrbeg7t/langman_2.gif">
 <br>
 <a href="https://travis-ci.org/themsaid/laravel-langman"><img src="https://travis-ci.org/themsaid/laravel-langman.svg?branch=master" alt="Build Status"></a>
 <a href="https://styleci.io/repos/55088784"><img src="https://styleci.io/repos/55088784/shield?style=flat" alt="StyleCI"></a>
@@ -23,7 +23,7 @@ Begin by installing the package through Composer. Run the following command in y
 $ composer require themsaid/laravel-langman
 ```
 
-Once done, add the following in the providers array of `config/app.php`:
+Once done, add the following line in your providers array of `config/app.php`:
 
 ```php
 Themsaid\Langman\LangmanServiceProvider::class
@@ -65,6 +65,14 @@ Brings only the translation of the `name` key in all languages.
 
 ---
 
+```
+php artisan langman:show users.name.first
+```
+
+Brings the translation of a nested key.
+
+---
+
 ```
 php artisan langman:show users.nam -c
 ```
@@ -104,7 +112,7 @@ asking you to give a translation for each, and finally save the given values to
 
 ```
 php artisan langman:trans users.name
-php artisan langman:trans users.name.en
+php artisan langman:trans users.name->en
 ```
 
 In the first case it'll ask you to give a translation for the given key in all languages, in the second case it'll ask you only
@@ -130,5 +138,4 @@ If you want a web interface to manage your language files instead, I recommend [
 by [Barry vd. Heuvel](https://github.com/barryvdh).
 
 ## TODO
-- Support nested translation lines in `langman:trans`, `langman:sync`, `langman:remove`, `langman:missing`.
 - Support vendor language files.

src/Commands/FindCommand.php

@@ -14,7 +14,7 @@ class FindCommand extends Command
      *
      * @var string
      */
-    protected $signature = 'langman:find {keyword}';
+    protected $signature = 'langman:find {keyword} {--package : Vendor Package name to search within.}';
 
     /**
      * The name and signature of the console command.
@@ -57,6 +57,10 @@ public function __construct(Manager $manager)
      */
     public function handle()
     {
+        if ($package = $this->option('package')) {
+            $this->manager->setPathToVendorPackage($package);
+        }
+
         $this->files = $this->manager->files();
 
         if (empty($this->files)) {

src/Manager.php

@@ -73,10 +73,14 @@ public function files()
             });
         });
 
-        // Hide files from "vendor" directory for now
-        $filesByFile = $filesByFile->filter(function ($value, $key) {
-            return ! Str::contains($key, ':');
-        });
+        // If the path does not contain "vendor" then we're looking at the
+        // main language files of the application, in this case we will
+        // neglect all vendor files.
+        if (! Str::contains($this->path, 'vendor')) {
+            $filesByFile = $filesByFile->filter(function ($value, $key) {
+                return ! Str::contains($key, ':');
+            });
+        }
 
         return $filesByFile->toArray();
     }
@@ -290,4 +294,15 @@ public function collectFromViews()
 
         return $output;
     }
+
+    /**
+     * Sets the path to a vendor package translation files.
+     *
+     * @param string $packageName
+     * @return void
+     */
+    public function setPathToVendorPackage($packageName)
+    {
+        $this->path = $this->path.'/vendor/'.$packageName;
+    }
 }

tests/FindCommandTest.php

@@ -43,4 +43,19 @@ public function testCommandOutputForFileWithNestedKeys()
         $this->assertRegExp('/user\.missing\.not_found(?:.*)user not found(?:.*)sp/', $this->consoleOutput());
         $this->assertNotContains('jarl_borg', $this->consoleOutput());
     }
+
+    public function testCommandOutputForPackage()
+    {
+        $this->createTempFiles([
+            'en' => ['user' => "<?php\n return ['weight' => 'weight'];", 'category' => ''],
+            'nl' => ['user' => '', 'category' => ''],
+            'vendor' => ['package' => ['en' => ['file' => "<?php\n return ['not_found' => 'file not found here'];"], 'sp' => ['file' => "<?php\n return ['not_found' => 'something'];"]]],
+        ]);
+
+        $this->artisan('langman:find', ['keyword' => 'not found', '--package' => 'package']);
+
+        $this->assertRegExp('/key(?:.*)en(?:.*)sp/', $this->consoleOutput());
+        $this->assertRegExp('/package::file\.not_found(?:.*)file not found here(?:.*)something/', $this->consoleOutput());
+        $this->assertNotContains('weight', $this->consoleOutput());
+    }
 }

tests/TestCase.php

@@ -42,7 +42,6 @@ public function createTempFiles($files = [])
 
                     foreach ($content as $subDir => $subContent) {
                         mkdir(__DIR__.'/temp/vendor/'.$file.'/'.$subDir);
-
                         foreach ($subContent as $subFile => $subsubContent) {
                             file_put_contents(__DIR__.'/temp/'.$dir.'/'.$file.'/'.$subDir.'/'.$subFile.'.php', $subsubContent);
                         }