improved user experience for novice user(including support for standalone aria2c and ytdl binaries);support for snap installation;close #19

.gitignore

@@ -5,3 +5,4 @@ build.sh
 test.php
 js/app*
 exclude
+bin

appinfo/application.php

@@ -56,7 +56,8 @@ class Application extends App
                 $container->query('UserId'),
                 \OC::$server->getL10N('ncdownloader'),
                 //\OC::$server->getRootFolder(),
-                $container->query('Aria2')
+                $container->query('Aria2'),
+                $container->query('Youtube')
             );
         });
 

lib/Controller/MainController.php

@@ -6,6 +6,8 @@ use OCA\NCDownloader\Tools\Aria2;
 use OCA\NCDownloader\Tools\Counters;
 use OCA\NCDownloader\Tools\DbHelper;
 use OCA\NCDownloader\Tools\Helper;
+use OCA\NCDownloader\Tools\Settings;
+use OCA\NCDownloader\Tools\Youtube;
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http\JSONResponse;
 use OCP\AppFramework\Http\TemplateResponse;
@@ -23,7 +25,7 @@ class MainController extends Controller
     private $aria2Opts;
     private $l10n;
 
-    public function __construct($appName, IRequest $request, $UserId, IL10N $IL10N, Aria2 $aria2)
+    public function __construct($appName, IRequest $request, $UserId, IL10N $IL10N, Aria2 $aria2, Youtube $youtube)
     {
         parent::__construct($appName, $request);
         $this->appName = $appName;
@@ -36,6 +38,9 @@ class MainController extends Controller
         $this->urlGenerator = \OC::$server->getURLGenerator();
         $this->dbconn = new DbHelper();
         $this->counters = new Counters($aria2, $this->dbconn, $UserId);
+        $this->youtube = $youtube;
+        $this->settings = new Settings($UserId);
+
     }
     /**
      * @NoAdminRequired
@@ -50,7 +55,13 @@ class MainController extends Controller
         $params = array();
         $params['aria2_running'] = $this->aria2->isRunning();
         $params['aria2_installed'] = $this->aria2->isInstalled();
-        $params['youtube_installed'] = (bool) Helper::findBinaryPath('youtube-dl');
+        $params['aria2_bin'] = $this->aria2->getBin();
+        $params['aria2_executable'] = $this->aria2->isExecutable();
+        $params['youtube_installed'] = $this->youtube->isInstalled();
+        $params['youtube_bin'] = $this->youtube->getBin();
+        $params['youtube_executable'] = $this->youtube->isExecutable();
+        $params['ncd_hide_errors'] = $this->settings->get("ncd_hide_errors");
+        
         $params['counter'] = $this->counters->getCounters();
         $params['settings_url'] = $this->urlGenerator->linkToRoute("settings.PersonalSettings.index", ['section' => 'ncdownloader']);
         $params['admin_settings_url'] = $this->urlGenerator->linkToRoute("settings.AdminSettings.index", ['section' => 'ncdownloader']);

lib/Controller/SettingsController.php

@@ -132,5 +132,4 @@ class SettingsController extends Controller
         }
         return ['message' => "Saved!"];
     }
-
 }

lib/Tools/Aria2.php

@@ -38,10 +38,10 @@ class Aria2
         );
         //turn keys in $options into variables
         extract($options);
-        if (isset($binary) && @is_executable($binary)) {
+        if (isset($binary) && $this->isExecutable($binary)) {
             $this->bin = $binary;
         } else {
-            $this->bin = Helper::findBinaryPath('aria2c');
+            $this->bin = Helper::findBinaryPath('aria2c', __DIR__ . "/../../bin/aria2c");
         }
         $this->setDownloadDir($dir);
         $this->setTorrentsDir($torrents_dir);
@@ -379,17 +379,27 @@ class Aria2
     }
     public function isInstalled()
     {
-        return (bool) (isset($this->bin) && @is_executable($this->bin));
+        return @is_file($this->bin);
     }
+    public function isExecutable()
+    {
+        return @is_executable($this->bin);
+    }
+
     public function isRunning()
     {
         $resp = $this->getSessionInfo();
         return (bool) $resp;
     }
+
+    public function getBin()
+    {
+        return $this->bin;
+    }
     public function stop()
     {
         $resp = $this->shutdown();
-        sleep(1);
+        sleep(3);
         return $resp ?? null;
     }
     private function confTemplate()

lib/Tools/Helper.php

@@ -202,6 +202,7 @@ class Helper
         if ($memcache->hasKey($program)) {
             return $memcache->get($program);
         }
+
         $dataPath = \OC::$server->getSystemConfig()->getValue('datadirectory');
         $paths = ['/usr/local/sbin', '/usr/local/bin', '/usr/sbin', '/usr/bin', '/sbin', '/bin', '/opt/bin', $dataPath . "/bin"];
         $result = $default;

lib/Tools/Youtube.php

@@ -18,6 +18,7 @@ class Youtube
     private $outTpl = "%(id)s-%(title).64s.%(ext)s";
     private $defaultDir = "/tmp/downloads";
     private $env = [];
+    private $bin;
 
     public function __construct(array $options)
     {
@@ -27,7 +28,7 @@ class Youtube
     public function init(array $options)
     {
         extract($options);
-        if (isset($binary) && @is_executable($binary)) {
+        if (isset($binary) && $this->isExecutable($binary)) {
             $this->bin = $binary;
         } else {
             $this->bin = Helper::findBinaryPath('youtube-dl', __DIR__ . "/../../bin/youtube-dl");
@@ -157,7 +158,7 @@ class Youtube
         $data = ['link' => $url];
         $process->run(function ($type, $buffer) use ($data, $process) {
             if (Process::ERR === $type) {
-               // $this->onError($buffer);
+                // $this->onError($buffer);
             } else {
                 $data['pid'] = $process->getPid();
                 $this->onOutput($buffer, $data);
@@ -235,7 +236,16 @@ class Youtube
     }
     public function isInstalled()
     {
-        return (bool) (isset($this->bin) && @is_executable($this->bin));
+        return @is_file($this->bin);
+    }
+    public function isExecutable()
+    {
+        return @is_executable($this->bin);
+    }
+
+    public function getBin()
+    {
+        return $this->bin;
     }
     public static function install()
     {

src/index.js

@@ -16,6 +16,7 @@ const basePath = "/apps/ncdownloader";
 
 window.addEventListener('DOMContentLoaded', function () {
 
+    helper.showErrors('[data-error-message]');
     // inputAction.run();
     updatePage.run();
     buttonActions.run();
@@ -60,10 +61,20 @@ window.addEventListener('DOMContentLoaded', function () {
         }).send();
     })
     eventHandler.add("click", "#app-navigation", "#search-download", helper.showDownload);
-    delegate('#ncdownloader-table-wrapper',
+    eventHandler.add("change", "#app-navigation", "#ncd-hide-errors", e => {
+        let data = {};
+        data["ncd_hide_errors"] = e.target.checked === "true";
+        console.log(data)
+        const url = helper.generateUrl(basePath + "/personal/save");
+        Http.getInstance(url).setData(data).setHandler(data => {
+            console.log(data);
+        }).send();
+    })
+    delegate('#app-ncdownloader-wrapper',
         { target: '[data-tippy-content]' }
     );
 
+
 });
 
 

src/utils/helper.js

@@ -178,6 +178,12 @@ const helper = {
     },
     getScrollTop() {
         return window.pageYOffset || document.documentElement.scrollTop || document.body.scrollTop || 0;
+    },
+    showErrors(target) {
+        let errors = document.querySelectorAll(target);
+        errors.forEach(element => {
+            helper.message(element.getAttribute('data-error-message'), 20000);
+        })
     }
 }
 

templates/Navigation.php

@@ -1,25 +1,38 @@
 <?php
-$aria2_running = $_['aria2_running'];
+extract($_);
-$youtube_installed = $_['youtube_installed'];
+$permission_error = sprintf("aria2 is installed but not executable.Please execute command sudo chmod 755 %s", $aria2_bin);
-$aria2_installed = $_['aria2_installed'];
+if ($youtube_installed && !$youtube_executable) {
-
+    $ytb_error = sprintf("youtube-dl is installed but not executable.Please execute command sudo chmod 755 %s", $youtube_bin);
+} else if (!$youtube_installed && !$youtube_executable) {
+    $ytb_error = "youtube-dl is not installed!";
+}
 ?>
 <div id="app-navigation">
+    <?php if ($ncd_hide_errors): ?>
+        <?php if (isset($ytb_error)): ?>
+            <div data-error-message="<?php print $l->t($ytb_error);?>"></div>
+        <?php endif;?>
+    <?php endif;?>
+
     <div class="app-navigation-new" id="search-download"  data-inputbox="form-input-wrapper">
         <button type="button" class="icon-add">
             <?php print($l->t('Download & Search'));?>
         </button>
     </div>
     <div class="app-navigation-new" id="start-aria2">
-        <?php if ($aria2_installed): ?>
+        <?php if ($aria2_installed && $aria2_executable): ?>
         <button type="button" class="icon-power"
             data-aria2="<?php $aria2_running ? print $l->t('on') : print $l->t('off');?>">
             <?php $aria2_running ? print $l->t('Stop Aria2') : print $l->t('Start Aria2');?>
         </button>
     </button>
+        <?php elseif ($aria2_installed && !$aria2_executable): ?>
+        <button type="button" class="icon-power notinstalled" data-error-message="<?php print $l->t($permission_error);?>">
+            <?php print $l->t("aria2c is installed but not executable");?>
+        </button>
         <?php else: ?>
         <button type="button" class="icon-power notinstalled">
-            <?php print $l->t('Aria2 is not installed!');?>
+            <?php print $l->t("aria2c is not installed!");?>
         </button>
         <?php endif;?>
     </div>

templates/settings/Settings.php

@@ -1,5 +1,9 @@
 <?php
 extract($_);
+$checked = '';
+if ($ncd_hide_errors) {
+    $checked = "checked";
+}
 ?>
 <div id="app-settings">
     <div id="app-settings-header">
@@ -11,6 +15,9 @@ extract($_);
     </div>
     <div id="app-settings-content">
         <ul id="ncdownloader-settings-collapsible-container">
+        <li class="ncdownloader-settings-item" data-tippy-content="check this to show or suppress errors">
+                <input class="checkbox" type="checkbox" value="<?php print($ncd_hide_errors);?>" <?php print($checked);?> id="ncd-hide-errors"><label for="ncd-hide-errors"><?php p($l->t('Hide Errors'));?></label>
+            </li>
             <li class="ncdownloader-settings-item">
                 <a href="<?php p($l->t($settings_url));?>" title="<?php p($l->t('Personal Settings'));?>" >
                     <?php p($l->t('Personal Settings'));?>