Fix compatibiliry with c++17 standard

Signed-off-by: Michele Calgaro <michele.calgaro@yahoo.it>
(cherry picked from commit ed66240370f342229f44f538414400fb48aa6400)

1 files changed, 20 insertions, 2 deletions

diff --git a/krusader/UserAction/expander.cpp b/krusader/UserAction/expander.cpp
index a037901..4b5a745 100644
--- a/krusader/UserAction/expander.cpp
+++ b/krusader/UserAction/expander.cpp
@@ -671,10 +671,19 @@ TagString exp_Copy::expFunc( const ListPanel*, const TagStringList& parameter, c
    // or transform(...) ?
    KURL dest = vfs::fromPathOrURL( parameter[1].string() );
 
-   if ( !dest.isValid() || find_if(src.constBegin(),src.constEnd(),not1(mem_fun_ref(&KURL::isValid) ))!=src.end()) {
+   if (!dest.isValid())
+   {
       setError(exp, Error(Error::S_FATAL,Error::C_ARGUMENT,i18n("Expander: invalid URL's in %_Copy(\"src\", \"dest\")%") ));
       return TQString();
    }
+   for (const KURL &url : src)
+   {
+     if (!url.isValid())
+     {
+      setError(exp, Error(Error::S_FATAL,Error::C_ARGUMENT,i18n("Expander: invalid URL's in %_Copy(\"src\", \"dest\")%") ));
+      return TQString();
+     }
+   }
 
    PreservingCopyJob::createCopyJob( PM_DEFAULT, src, dest, TDEIO::CopyJob::Copy, false, true );
 
@@ -702,9 +711,18 @@ TagString exp_Move::expFunc( const ListPanel*, const TagStringList& parameter, c
    // or transform(...) ?
    KURL dest = vfs::fromPathOrURL( parameter[1].string() );
 
-   if ( !dest.isValid() || find_if(src.constBegin(),src.constEnd(),not1(mem_fun_ref(&KURL::isValid) ))!=src.end()) {
+   if (!dest.isValid())
+   {
+      setError(exp, Error(Error::S_FATAL,Error::C_ARGUMENT,i18n("Expander: invalid URL's in %_Move(\"src\", \"dest\")%") ));
+      return TQString();
+   }
+   for (const KURL &url : src)
+   {
+     if (!url.isValid())
+     {
       setError(exp, Error(Error::S_FATAL,Error::C_ARGUMENT,i18n("Expander: invalid URL's in %_Move(\"src\", \"dest\")%") ));
       return TQString();
+     }
    }
 
    PreservingCopyJob::createCopyJob( PM_DEFAULT, src, dest, TDEIO::CopyJob::Move, false, true );