upd tests, add notes to readme

Author: Gemorroj

README.md

@@ -21,6 +21,13 @@
 - **7-zip >= 7.30 (p7zip >= 9.38)**
 
 
+### Примечания:
+
+ - Список файлов/директорий отображается всегда в кодировке UTF-8
+ - При указании файлов/директорий для распаковки, их имена нужно указывать в кидировке текущей файловой системы
+ - При распаковке архива, имена файлов/директорий запишутся в кодировке текущей файловой системы
+
+
 ### Установка через composer:
 
 - Добавьте проект в ваш файл composer.json:

Tests/Archive7z/Archive7zTest.php

@@ -23,6 +23,16 @@ protected function setUp()
         $this->mock = $this->getMock('Archive7z\Archive7z', null, array('fake.7z', $this->cliPath));
     }
 
+
+    protected function getCurrentFilesystemEncoding()
+    {
+        if (stripos(PHP_OS, 'WIN') !== false) { // windows
+            return 'Windows-1251';
+        }
+        return exec('locale charmap');
+    }
+
+
     protected function tearDown()
     {
         $this->cleanDir($this->tmpDir);
@@ -110,7 +120,7 @@ public function testExtract()
     public function testExtractCyrillic()
     {
         $dirCyrillic = $this->tmpDir . '/папка';
-        $chavezFile = iconv('UTF-8', 'Windows-1251', 'чавес.jpg');
+        $chavezFile = iconv('UTF-8', $this->getCurrentFilesystemEncoding(), 'чавес.jpg');
 
         if (!mkdir($dirCyrillic)) {
             self::markTestIncomplete('Cant create cyrillic directory.');
@@ -231,9 +241,9 @@ public function testExtractEntryOverwrite()
     }
 
 
-    public function testExtractEntryUnicode()
+    public function testExtractEntryCyrillic()
     {
-        $file = iconv('UTF-8', 'Windows-1251', 'чавес.jpg');
+        $file = iconv('UTF-8', $this->getCurrentFilesystemEncoding(), 'чавес.jpg');
         $obj = new Archive7z($this->fixturesDir . '/test.7z', $this->cliPath);
         $obj->setOutputDirectory($this->tmpDir);
         $obj->extractEntry($file);
@@ -396,4 +406,25 @@ public function testRenameEntryPasswd()
         $resultDest = $obj->getEntry('test' . DIRECTORY_SEPARATOR . 'newTest.txt');
         self::assertInstanceOf('Archive7z\Entry', $resultDest);
     }
+
+
+    public function testChangeSystemLocale()
+    {
+        $file = iconv('UTF-8', $this->getCurrentFilesystemEncoding(), 'чавес.jpg');
+        $obj = new Archive7z($this->fixturesDir . '/test.7z', $this->cliPath);
+        $obj->setChangeSystemLocale(true);
+        $obj->setOutputDirectory($this->tmpDir);
+        $obj->extractEntry($file);
+
+        self::assertFileExists($this->tmpDir . '/' . $file);
+    }
+
+    public function testChangeSystemLocaleFail()
+    {
+        $new = new Archive7z($this->tmpDir . '/test.7z', $this->cliPath);
+        $new->setChangeSystemLocale(true);
+        $this->setExpectedException('Archive7z\Exception');
+        $new->getContent('file.txt');
+    }
+
 }

src/Archive7z/Archive7z.php

@@ -90,7 +90,11 @@ class Archive7z
     /**
      * @var string
      */
-    protected $systemLocale = 'en_US.utf8';
+    protected $systemLocaleNix = 'en_US.utf8';
+    /**
+     * @var string
+     */
+    protected $systemLocaleWin = '65001';
 
 
     /**
@@ -532,10 +536,10 @@ public function isValid()
      */
     protected function execute($cmd)
     {
-        if (!$this->getChangeSystemLocale() || $this->isOsWin()) {
+        if (!$this->getChangeSystemLocale()) {
             $out = $this->exec($cmd);
         } else {
-            $out = $this->execLang($cmd);
+            $out = $this->execLocale($cmd);
         }
 
         return $out;
@@ -564,9 +568,13 @@ protected function exec($cmd)
      * @return array
      * @throws Exception
      */
-    protected function execLang($cmd)
+    protected function execLocale($cmd)
     {
-        return $this->exec('LANG=' . escapeshellarg($this->systemLocale) . ' ' . $cmd);
+        if ($this->isOsWin()) {
+            return $this->exec('chcp ' . escapeshellarg($this->systemLocaleWin) . ' & ' . $cmd);
+        } else {
+            return $this->exec('LANG=' . escapeshellarg($this->systemLocaleNix) . ' ' . $cmd);
+        }
     }