Fix tdf#97500 Reinstate missing file error dialog on WebDAV

Bug introduced with my fix to tdf#96669 in commit
5fc2910fc872bbd1184aaab7c842dff593d2449b.

Reverted previous fix and write a new version in fpicker instead.

On WebDAV better check if the stream opens first, then check
IsDocument property, because it may be a folder name.

Change-Id: I72ce728329e4194080db6fa4cc4d98fecf7672e9
Reviewed-on: https://gerrit.libreoffice.org/22214
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>

1 files changed, 5 insertions, 21 deletions

diff --git a/ucb/source/ucp/webdav-neon/webdavcontent.cxx b/ucb/source/ucp/webdav-neon/webdavcontent.cxx
index 4924e02ad845..b68dc64f3119 100644
--- a/ucb/source/ucp/webdav-neon/webdavcontent.cxx
+++ b/ucb/source/ucp/webdav-neon/webdavcontent.cxx
@@ -865,7 +865,6 @@ void Content::addProperty( const ucb::PropertyCommandArgument& aCmdArg,
                     switch ( eType )
                     {
                     case UNKNOWN:
-                    case NOT_FOUND:
                     case DAV:
                         throw lang::IllegalArgumentException();
 
@@ -954,7 +953,6 @@ void Content::removeProperty( const OUString& Name,
                     switch ( eType )
                     {
                         case UNKNOWN:
-                        case NOT_FOUND:
                         case DAV:
                             throw beans::UnknownPropertyException();
 
@@ -1444,7 +1442,7 @@ uno::Reference< sdbc::XRow > Content::getPropertyValues(
         NeonUri aUri( xResAccess->getURL() );
         aUnescapedTitle = aUri.GetPathBaseNameUnescaped();
 
-        if ( eType == UNKNOWN || eType == NOT_FOUND )
+        if ( eType == UNKNOWN )
         {
             xProps.reset( new ContentProperties( aUnescapedTitle ) );
         }
@@ -1475,20 +1473,10 @@ uno::Reference< sdbc::XRow > Content::getPropertyValues(
                 "IsFolder",
                 uno::makeAny( false ),
                 true );
-            if ( eType == NOT_FOUND )
-            {
-                xProps->addProperty(
-                "IsDocument",
-                uno::makeAny( false ),
-                true );
-            }
-            else
-            {
-                xProps->addProperty(
+            xProps->addProperty(
                 "IsDocument",
                 uno::makeAny( true ),
                 true );
-            }
         }
     }
     else
@@ -2912,7 +2900,7 @@ Content::ResourceType Content::resourceTypeForLocks(
         }
     }
     osl::MutexGuard g(m_aMutex);
-    if ( m_eResourceTypeForLocks == UNKNOWN || m_eResourceTypeForLocks == NOT_FOUND )
+    if (m_eResourceTypeForLocks == UNKNOWN)
     {
         m_eResourceTypeForLocks = eResourceTypeForLocks;
     }
@@ -3450,7 +3438,7 @@ Content::ResourceType Content::getResourceType(
 {
     {
         osl::MutexGuard g(m_aMutex);
-        if ( m_eResourceType != UNKNOWN && m_eResourceType != NOT_FOUND ) {
+        if (m_eResourceType != UNKNOWN) {
             return m_eResourceType;
         }
     }
@@ -3499,10 +3487,6 @@ Content::ResourceType Content::getResourceType(
         {
             rResAccess->resetUri();
 
-            if ( e.getStatus() == SC_NOT_FOUND )
-            {
-                eResourceType = NOT_FOUND;
-            }
             if ( e.getStatus() == SC_METHOD_NOT_ALLOWED )
             {
                 // Status SC_METHOD_NOT_ALLOWED is a safe indicator that the
@@ -3526,7 +3510,7 @@ Content::ResourceType Content::getResourceType(
     }
 
     osl::MutexGuard g(m_aMutex);
-    if ( m_eResourceType == UNKNOWN || m_eResourceType == NOT_FOUND ) {
+    if (m_eResourceType == UNKNOWN) {
         m_eResourceType = eResourceType;
     } else {
         SAL_WARN_IF(