[tx-robot] updated from transifex

The other one was not working anyway.

CMakeLists.txt

@@ -4,8 +4,8 @@ project(mirall)
 set(PACKAGE "mirall")
 set( CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake/modules )
 
-if ( EXISTS ${CMAKE_SOURCE_DIR}/OEM.cmake )
-     include ( ${CMAKE_SOURCE_DIR}/OEM.cmake )
+if ( DEFINED OEM_THEME_DIR AND EXISTS ${OEM_THEME_DIR}/OEM.cmake )
+     include ( ${OEM_THEME_DIR}/OEM.cmake )
 else ()
      include ( ${CMAKE_SOURCE_DIR}/OWNCLOUD.cmake )
 endif()
@@ -15,6 +15,22 @@ configure_file( ${CMAKE_SOURCE_DIR}/src/mirall/version.h.in "${CMAKE_CURRENT_BIN
 include_directories(BEFORE "${CMAKE_CURRENT_BINARY_DIR}/src/mirall/")
 
 include(GNUInstallDirs)
+
+include(GetGitRevisionDescription)
+get_git_head_revision(GIT_REFSPEC GIT_SHA1)
+
+# if we cannot get it from git, directly try .tag (packages)
+# this will work if the tar balls have been properly created
+# via git-archive.
+if (${GIT_SHA1} STREQUAL "GITDIR-NOTFOUND")
+    file(READ ${CMAKE_SOURCE_DIR}/.tag sha1_candidate)
+    string(REPLACE "\n" "" sha1_candidate ${sha1_candidate})
+    if (NOT ${sha1_candidate} STREQUAL "$Format:%H$")
+        message("${sha1_candidate}")
+        set (GIT_SHA1 "${sha1_candidate}")
+    endif()
+endif()
+
 ## stupid, we should upstream this
 if("${CMAKE_INSTALL_PREFIX}" STREQUAL "/usr" AND NOT CMAKE_INSTALL_SYSCONFDIR)
     set(CMAKE_INSTALL_SYSCONFDIR "/etc")
@@ -37,13 +53,20 @@ else()
 endif()
 #####
 
+#### find libs
 find_package(Qt4 4.6.0 COMPONENTS QtCore QtGui QtXml QtNetwork QtTest REQUIRED )
 find_package(Csync)
+if(UNIX)
+find_package(INotify REQUIRED)
+else()
 find_package(INotify)
+endif()
 find_package(Sphinx)
 find_package(PdfLatex)
+find_package(QtKeychain)
 
 set(WITH_CSYNC CSYNC_FOUND)
+set(WITH_QTKEYCHAIN ${QTKEYCHAIN_FOUND})
 set(USE_INOTIFY ${INOTIFY_FOUND})
 
 configure_file(config.h.in ${CMAKE_CURRENT_BINARY_DIR}/config.h)

CONTRIBUTING.md

@@ -0,0 +1,48 @@
+## Submitting Desktop Client issues
+
+If you have questions about how to use the ownCloud Desktop Client, please
+direct these to the [mailing list][mailinglist] or our [forum][forum].
+We are also available on [IRC][irc].
+
+### Guidelines
+* Report the issue using our [template][template], it includes all the
+  informations we need to track down the issue.  
+* This repository is *only* for issues within the ownCloud desktop client. 
+  Issues in other compontents should be reported in their own repositores: 
+  - [ownCloud server](https://github.com/owncloud/core/issues)
+  - [ownCloud apps](https://github.com/owncloud/apps/issues) (e.g. Calendar,
+    Contacts...)
+  - [Android client](https://github.com/owncloud/android/issues)
+  - [iOS client](https://github.com/owncloud/ios-issues/issues)
+* Search the existing issues first, it's likely that your issue was already
+  reported.
+
+If your issue appears to be a bug, and hasn't been reported, open a new issue.
+
+Help us to maximize the effort we can spend fixing issues and adding new
+features, by not reporting duplicate issues.
+
+[template]: https://raw.github.com/owncloud/mirall/master/issue_template.md
+[mailinglist]: https://mail.kde.org/mailman/listinfo/owncloud
+[forum]: http://forum.owncloud.org/
+[irc]: http://webchat.freenode.net/?channels=owncloud&uio=d4
+
+## Contributing to Source Code
+
+Thanks for wanting to contribute source code to ownCloud. That's great!
+
+Before we're able to merge your code to mirall, you need to sign
+our [Contributor Agreement][agreement].
+
+Please read the [Desktop Client Manual][mirallman] and the [Developer
+Manuals][devmanual] to get useful infos like how to create your first
+application or how to test the ownCloud code with phpunit.
+
+[agreement]: http://owncloud.org/about/contributor-agreement/
+[devmanual]: http://owncloud.org/dev/
+[mirallman]: http://doc.owncloud.org/desktop/1.1/
+
+## Translations
+Please submit translations via [Transifex][transifex].
+
+[transifex]: https://www.transifex.com/projects/p/owncloud/

CPackOptions.cmake.in

@@ -9,9 +9,10 @@ endif(CPACK_GENERATOR MATCHES "NSIS")
 
 set( CMAKE_SOURCE_DIR  @CMAKE_SOURCE_DIR@ )
 set( CMAKE_BINARY_DIR  @CMAKE_BINARY_DIR@ )
+set( OEM_THEME_DIR     @OEM_THEME_DIR@ )
 
-if ( EXISTS "${CMAKE_SOURCE_DIR}/OEM.cmake" )
-     include ( "${CMAKE_SOURCE_DIR}/OEM.cmake" )
+if ( DEFINED OEM_THEME_DIR AND EXISTS ${OEM_THEME_DIR}/OEM.cmake )
+     include ( ${OEM_THEME_DIR}/OEM.cmake )
 else ()
      include ( "${CMAKE_SOURCE_DIR}/OWNCLOUD.cmake" )
 endif()

ChangeLog

@@ -1,6 +1,26 @@
 ChangeLog
 =========
-version 1.1.2 (release 2012-11- ), csync 0.60.2 required
+
+version 1.2.0 (release 2013-01-24 ), csync 0.70.2 required
+  * [GUI] New status dialog to show a detailed list of synced files.
+  * [GUI] New tray notifications about synced files.
+  * [GUI] New platform specific icon set.
+  * [App] Using cross platform QtKeychain library to store credentials crypted.
+  * [App] Use cross platform notification for changes in the local file system rather than regular poll.
+  * [Fixes] Improved SSL Certificate handling and SSL fixes troughout syncing.
+  * [Fixes] Fixed proxy authentication.
+  * [Fixes] Allow brackets in folder name alias.
+  * [Fixes] Lots of other minor fixes.
+  * [Platform] cmake fixes.
+  * [Platform] Improved, more detailed error reporting.
+
+version 1.1.4 (release 2012-12-19 ), csync 0.60.4 required
+  * No changes to mirall, only csync fixes.
+
+version 1.1.3 (release 2012-11-30 ), csync 0.60.3 required
+  * No changes to mirall, only csync fixes.
+
+version 1.1.2 (release 2012-11-26 ), csync 0.60.2 required
   * [Fixes] Allow to properly cancel the password dialog.
   * [Fixes] Share folder name correctly percent encoded with old Qt
             4.6 builds ie. Debian.
@@ -18,18 +38,18 @@ version 1.1.2 (release 2012-11- ), csync 0.60.2 required
   * [Platform] Fix Mac building
 
 version 1.1.1 (release 2012-10-18), csync 0.60.1 required
-  * [GUI]   Allow changing folder name in single folder mode 
+  * [GUI]   Allow changing folder name in single folder mode
   * [GUI]   Windows: Add license to installer
   * [GUI]   owncloud --logwindow will bring up the log window
             in an already running instance
   * [Fixes] Make sure SSL errors are always handled
-  * [Fixes] Allow special characters in folder alias 
-  * [Fixes] Proper workaround for Menu bug in Ubuntu 
+  * [Fixes] Allow special characters in folder alias
+  * [Fixes] Proper workaround for Menu bug in Ubuntu
   * [Fixes] csync: Fix improper memory cleanup which could
             cause memory leaks and crashes
-  * [Fixes] csync: Fix memory leak 
-  * [Fixes] csync: Allow single quote (') in file names 
-  * [Fixes] csync: Remove stray temporary files 
+  * [Fixes] csync: Fix memory leak
+  * [Fixes] csync: Allow single quote (') in file names
+  * [Fixes] csync: Remove stray temporary files
 
 version 1.1.0 (release 2012-10-10), csync 0.60.0 required
   * [GUI]   Added an about dialog
@@ -53,7 +73,7 @@ version 1.1.0 (release 2012-10-10), csync 0.60.0 required
   * [Platform] csync conf file and database were moved to the users app data
                directory, away from the .csync dir.
   *         Renamed exclude.lst to sync-exclude.lst and moved it to
-            /etc/appName()/ for more clean packaging. From the user path, 
+            /etc/appName()/ for more clean packaging. From the user path,
 	    still exclude.lst is read if sync-exclude.lst is not existing.
   *         Placed custom.ini with customization options to /etc/appName()
 
@@ -62,7 +82,7 @@ version 1.0.5 (release 2012-08-14), csync 0.50.8 required
 
 version 1.0.4 (release 2012-08-10), csync 0.50.8 required
   * [APP] ownCloud is now a single instance app, can not start twice any more.
-  * [APP] Proxy support 
+  * [APP] Proxy support
   * [APP] Handle HTTP redirection correctly, note new url.
   * [APP] More relaxed handling of read only directories in the sync paths.
   * [APP] Started to split off a library with sync functionality, eg for KDE
@@ -73,13 +93,13 @@ version 1.0.4 (release 2012-08-10), csync 0.50.8 required
   * [GUI] Removed Log Window Button, log available through command line.
   * [GUI] Proxy configuration dialog added.
   * [GUI] Added Translations to languages Slovenian, Polish, Catalan,
-          Portuguese (Brazil), German, Greek, Spanish, Czech, Italian, Slovak, 
-	  French, Russian, Japanese, Swedish, Portuguese (Portugal) 
+          Portuguese (Brazil), German, Greek, Spanish, Czech, Italian, Slovak,
+	  French, Russian, Japanese, Swedish, Portuguese (Portugal)
 	  all with translation rate >90%.
   * [Fixes] Loading of self signed certs into Networkmanager (#oc-843)
   * [Fixes] Win32: Handle SSL dll loading correctly.
   * [Fixes] Many other small fixes and improvements.
-  
+
 version 1.0.3 (release 2012-06-19), csync 0.50.7 required
   * [GUI] Added a log window which catches the logging if required and
           allows to save for information.
@@ -88,7 +108,7 @@ version 1.0.3 (release 2012-06-19), csync 0.50.7 required
   * [Fixes] Do not use csync database files from a sync before.
   * [Fixes] In Connection wizard, write the final config onyl if
             the user really accepted. Also remove the former database.
-  * [Fixes] More user expected behaviour deletion of sync folder local 
+  * [Fixes] More user expected behaviour deletion of sync folder local
             and remote.
   * [Fixes] Allow special characters in the sync directory names
   * [Fixes] Win32: Fixed directory removal with special character dirs.

OWNCLOUD.cmake

@@ -1,7 +1,10 @@
 set( APPLICATION_SHORTNAME  "owncloud" )
 set( APPLICATION_NAME       "ownCloud" )
 set( APPLICATION_EXECUTABLE "owncloud"        )
-set( APPLICATION_DOMAIN     "owncloud.org"    )
-set( APPLICATION_VENDOR     "ownCloud.org"    )
+set( APPLICATION_DOMAIN     "owncloud.com"    )
+set( APPLICATION_VENDOR     "ownCloud, Inc"   )
 set( THEME_CLASS            "ownCloudTheme"   )
-set( APPLICATION_REV_DOMAIN "org.owncloud.desktopclient" )
+set( APPLICATION_REV_DOMAIN "com.owncloud.desktopclient" )
+set( WIN_SETUP_BITMAP_PATH  "${CMAKE_SOURCE_DIR}/admin/win/nsi" )
+# set( THEME_INCLUDE          "${OEM_THEME_DIR}/mytheme.h" )
+# set( APPLICATION_LICENSE    "${OEM_THEME_DIR}/license.txt )

OwnCloudCPack.cmake

@@ -2,8 +2,8 @@ include( InstallRequiredSystemLibraries )
 
 set( CPACK_PACKAGE_CONTACT  "Dominik Schmidt <domme@tomahawk-player.org>" )
 
-if ( EXISTS "${CMAKE_SOURCE_DIR}/OEM.cmake" )
-     include ( "${CMAKE_SOURCE_DIR}/OEM.cmake" )
+if ( DEFINED OEM_THEME_DIR AND EXISTS ${OEM_THEME_DIR}/OEM.cmake )
+     include ( "${OEM_THEME_DIR}/OEM.cmake" )
 else ()
      include ( "${CMAKE_SOURCE_DIR}/OWNCLOUD.cmake" )
 endif()

VERSION.cmake

@@ -1,5 +1,5 @@
 set( VERSION_MAJOR 1 )
-set( VERSION_MINOR 1 )
-set( VERSION_PATCH 2 )
-set( VERSION  ${VERSION_MAJOR}.${VERSION_MINOR}.${VERSION_PATCH}rc )
+set( VERSION_MINOR 2 )
+set( VERSION_PATCH 0 )
+set( VERSION  ${VERSION_MAJOR}.${VERSION_MINOR}.${VERSION_PATCH})
 

admin/win/Toolchain-mingw32-openSUSE.cmake

@@ -32,3 +32,7 @@ SET(QT_MOC_EXECUTABLE       ${MINGW_PREFIX}-moc)
 SET(QT_RCC_EXECUTABLE       ${MINGW_PREFIX}-rcc)
 SET(QT_UIC_EXECUTABLE       ${MINGW_PREFIX}-uic)
 SET(QT_LRELEASE_EXECUTABLE  ${MINGW_PREFIX}-lrelease)
+
+# neon config
+SET(NEON_CONFIG_EXECUTABLE  ${CMAKE_FIND_ROOT_PATH}/bin/neon-config)
+# /usr/i686-w64-mingw32/sys-root/mingw/bin/neon-config

cmake/modules/FindCsync.cmake

@@ -1,28 +1,28 @@
-IF(EXISTS "${CMAKE_SOURCE_DIR}/../csync/src/csync.h")
-  SET(CSYNC_INCLUDE_PATH ${CMAKE_SOURCE_DIR}/../csync/src/)
-ELSE()
-  FIND_PATH(CSYNC_INCLUDE_PATH NAMES csync.h HINTS $ENV{CSYNC_DIR} )
-ENDIF()
 
-IF( DEFINED CSYNC_LIBRARY_PATH AND DEFINED CSYNC_INCLUDE_PATH )
-  IF( WIN32 )
-    SET(CSYNC_LIBRARY ${CSYNC_LIBRARY_PATH}/src/libocsync.dll)
-  ELSEIF ( APPLE )
-    SET(CSYNC_LIBRARY ${CSYNC_LIBRARY_PATH}/src/libocsync.dylib)
+IF( DEFINED CSYNC_BUILD_PATH )
+  SET(CSYNC_LIBRARY_PATH ${CSYNC_BUILD_PATH})
+  SET(CSYNC_BINARY_DIR ${CSYNC_BUILD_PATH})
+
+  IF(WIN32)
+    SET(CSYNC_LIBRARY ${CSYNC_BUILD_PATH}/src/libocsync.dll)
+  ELSEIF( APPLE )
+    SET(CSYNC_LIBRARY ${CSYNC_BUILD_PATH}/src/libocsync.dylib)
   ELSE()
-    SET(CSYNC_LIBRARY ${CSYNC_LIBRARY_PATH}/src/libocsync.so)
-  ENDIF( )
-  SET(CSYNC_BINARY_DIR ${CSYNC_LIBRARY_PATH})
+    SET(CSYNC_LIBRARY ${CSYNC_BUILD_PATH}/src/libocsync.so)
+  ENDIF()
 ELSE()
   FIND_LIBRARY(CSYNC_LIBRARY NAMES ocsync HINTS $ENV{CSYNC_DIR})
 ENDIF()
 
+IF(NOT DEFINED CSYNC_INCLUDE_PATH)
+  FIND_PATH(CSYNC_INCLUDE_PATH NAMES csync.h HINTS $ENV{CSYNC_DIR} )
+ENDIF()
+
 SET(CSYNC_INCLUDE_DIR ${CSYNC_INCLUDE_PATH})
 
 # handle the QUIETLY and REQUIRED arguments and set CSYNC_FOUND to TRUE if
 # all listed variables are TRUE
 include(FindPackageHandleStandardArgs)
-FIND_PACKAGE_HANDLE_STANDARD_ARGS(Csync DEFAULT_MSG
-  CSYNC_LIBRARY CSYNC_INCLUDE_PATH)
+FIND_PACKAGE_HANDLE_STANDARD_ARGS(Csync DEFAULT_MSG CSYNC_LIBRARY CSYNC_INCLUDE_PATH)
 
 MARK_AS_ADVANCED( CSYNC_INCLUDE_PATH CSYNC_LIBRARY )

cmake/modules/FindINotify.cmake

@@ -4,12 +4,12 @@
 # also defined, but not for general use are
 #  INOTIFY_LIBRARY, where to find the inotify library.
 
-find_path(INOTIFY_INCLUDE_DIR sys/inotify.h)
+find_path(INOTIFY_INCLUDE_DIR sys/inotify.h 
+          HINTS /usr/include/${CMAKE_LIBRARY_ARCHITECTURE})
 mark_as_advanced(INOTIFY_INCLUDE_DIR)
 
-
-# handle the QUIETLY and REQUIRED arguments and set INOTIFY_FOUND to TRUE if
 # all listed variables are TRUE
+# handle the QUIETLY and REQUIRED arguments and set INOTIFY_FOUND to TRUE if
 include(FindPackageHandleStandardArgs)
 find_package_handle_standard_args(INOTIFY DEFAULT_MSG INOTIFY_INCLUDE_DIR)
 

cmake/modules/FindQtKeychain.cmake

@@ -0,0 +1,18 @@
+# - Try to find QtKeyChain
+# Once done this will define
+#  QTKEYCHAIN_FOUND - System has QtKeyChain
+#  QTKEYCHAIN_INCLUDE_DIRS - The QtKeyChain include directories
+#  QTKEYCHAIN_LIBRARIES - The libraries needed to use QtKeyChain
+#  QTKEYCHAIN_DEFINITIONS - Compiler switches required for using LibXml2
+
+find_path(QTKEYCHAIN_INCLUDE_DIR qtkeychain/keychain.h)
+
+find_library(QTKEYCHAIN_LIBRARY NAMES libqtkeychain qtkeychain)
+
+include(FindPackageHandleStandardArgs)
+# handle the QUIETLY and REQUIRED arguments and set QTKEYCHAIN_FOUND to TRUE
+# if all listed variables are TRUE
+find_package_handle_standard_args(QtKeyChain  DEFAULT_MSG
+	QTKEYCHAIN_LIBRARY QTKEYCHAIN_INCLUDE_DIR)
+
+mark_as_advanced(QTKEYCHAIN_INCLUDE_DIR QTKEYCHAIN_LIBRARY )

cmake/modules/GetGitRevisionDescription.cmake

@@ -0,0 +1,123 @@
+# - Returns a version string from Git
+#
+# These functions force a re-configure on each git commit so that you can
+# trust the values of the variables in your build system.
+#
+#  get_git_head_revision(<refspecvar> <hashvar> [<additional arguments to git describe> ...])
+#
+# Returns the refspec and sha hash of the current head revision
+#
+#  git_describe(<var> [<additional arguments to git describe> ...])
+#
+# Returns the results of git describe on the source tree, and adjusting
+# the output so that it tests false if an error occurs.
+#
+#  git_get_exact_tag(<var> [<additional arguments to git describe> ...])
+#
+# Returns the results of git describe --exact-match on the source tree,
+# and adjusting the output so that it tests false if there was no exact
+# matching tag.
+#
+# Requires CMake 2.6 or newer (uses the 'function' command)
+#
+# Original Author:
+# 2009-2010 Ryan Pavlik <rpavlik@iastate.edu> <abiryan@ryand.net>
+# http://academic.cleardefinition.com
+# Iowa State University HCI Graduate Program/VRAC
+#
+# Copyright Iowa State University 2009-2010.
+# Distributed under the Boost Software License, Version 1.0.
+# (See accompanying file LICENSE_1_0.txt or copy at
+# http://www.boost.org/LICENSE_1_0.txt)
+
+if(__get_git_revision_description)
+	return()
+endif()
+set(__get_git_revision_description YES)
+
+# We must run the following at "include" time, not at function call time,
+# to find the path to this module rather than the path to a calling list file
+get_filename_component(_gitdescmoddir ${CMAKE_CURRENT_LIST_FILE} PATH)
+
+function(get_git_head_revision _refspecvar _hashvar)
+	set(GIT_PARENT_DIR "${CMAKE_SOURCE_DIR}")
+	set(GIT_DIR "${GIT_PARENT_DIR}/.git")
+	while(NOT EXISTS "${GIT_DIR}")	# .git dir not found, search parent directories
+		set(GIT_PREVIOUS_PARENT "${GIT_PARENT_DIR}")
+		get_filename_component(GIT_PARENT_DIR ${GIT_PARENT_DIR} PATH)
+		if(GIT_PARENT_DIR STREQUAL GIT_PREVIOUS_PARENT)
+			# We have reached the root directory, we are not in git
+			set(${_refspecvar} "GITDIR-NOTFOUND" PARENT_SCOPE)
+			set(${_hashvar} "GITDIR-NOTFOUND" PARENT_SCOPE)
+			return()
+		endif()
+		set(GIT_DIR "${GIT_PARENT_DIR}/.git")
+	endwhile()
+	set(GIT_DATA "${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/git-data")
+	if(NOT EXISTS "${GIT_DATA}")
+		file(MAKE_DIRECTORY "${GIT_DATA}")
+	endif()
+
+	if(NOT EXISTS "${GIT_DIR}/HEAD")
+		return()
+	endif()
+	set(HEAD_FILE "${GIT_DATA}/HEAD")
+	configure_file("${GIT_DIR}/HEAD" "${HEAD_FILE}" COPYONLY)
+
+	configure_file("${_gitdescmoddir}/GetGitRevisionDescription.cmake.in"
+		"${GIT_DATA}/grabRef.cmake"
+		@ONLY)
+	include("${GIT_DATA}/grabRef.cmake")
+
+	set(${_refspecvar} "${HEAD_REF}" PARENT_SCOPE)
+	set(${_hashvar} "${HEAD_HASH}" PARENT_SCOPE)
+endfunction()
+
+function(git_describe _var)
+	if(NOT GIT_FOUND)
+		find_package(Git QUIET)
+	endif()
+	get_git_head_revision(refspec hash)
+	if(NOT GIT_FOUND)
+		set(${_var} "GIT-NOTFOUND" PARENT_SCOPE)
+		return()
+	endif()
+	if(NOT hash)
+		set(${_var} "HEAD-HASH-NOTFOUND" PARENT_SCOPE)
+		return()
+	endif()
+
+	# TODO sanitize
+	#if((${ARGN}" MATCHES "&&") OR
+	#	(ARGN MATCHES "||") OR
+	#	(ARGN MATCHES "\\;"))
+	#	message("Please report the following error to the project!")
+	#	message(FATAL_ERROR "Looks like someone's doing something nefarious with git_describe! Passed arguments ${ARGN}")
+	#endif()
+
+	#message(STATUS "Arguments to execute_process: ${ARGN}")
+
+	execute_process(COMMAND
+		"${GIT_EXECUTABLE}"
+		describe
+		${hash}
+		${ARGN}
+		WORKING_DIRECTORY
+		"${CMAKE_SOURCE_DIR}"
+		RESULT_VARIABLE
+		res
+		OUTPUT_VARIABLE
+		out
+		ERROR_QUIET
+		OUTPUT_STRIP_TRAILING_WHITESPACE)
+	if(NOT res EQUAL 0)
+		set(out "${out}-${res}-NOTFOUND")
+	endif()
+
+	set(${_var} "${out}" PARENT_SCOPE)
+endfunction()
+
+function(git_get_exact_tag _var)
+	git_describe(out --exact-match ${ARGN})
+	set(${_var} "${out}" PARENT_SCOPE)
+endfunction()

cmake/modules/GetGitRevisionDescription.cmake.in

@@ -0,0 +1,38 @@
+# 
+# Internal file for GetGitRevisionDescription.cmake
+#
+# Requires CMake 2.6 or newer (uses the 'function' command)
+#
+# Original Author:
+# 2009-2010 Ryan Pavlik <rpavlik@iastate.edu> <abiryan@ryand.net>
+# http://academic.cleardefinition.com
+# Iowa State University HCI Graduate Program/VRAC
+#
+# Copyright Iowa State University 2009-2010.
+# Distributed under the Boost Software License, Version 1.0.
+# (See accompanying file LICENSE_1_0.txt or copy at
+# http://www.boost.org/LICENSE_1_0.txt)
+
+set(HEAD_HASH)
+
+file(READ "@HEAD_FILE@" HEAD_CONTENTS LIMIT 1024)
+
+string(STRIP "${HEAD_CONTENTS}" HEAD_CONTENTS)
+if(HEAD_CONTENTS MATCHES "ref")
+	# named branch
+	string(REPLACE "ref: " "" HEAD_REF "${HEAD_CONTENTS}")
+	if(EXISTS "@GIT_DIR@/${HEAD_REF}")
+		configure_file("@GIT_DIR@/${HEAD_REF}" "@GIT_DATA@/head-ref" COPYONLY)
+	elseif(EXISTS "@GIT_DIR@/logs/${HEAD_REF}")
+		configure_file("@GIT_DIR@/logs/${HEAD_REF}" "@GIT_DATA@/head-ref" COPYONLY)
+		set(HEAD_HASH "${HEAD_REF}")
+	endif()
+else()
+	# detached HEAD
+	configure_file("@GIT_DIR@/HEAD" "@GIT_DATA@/head-ref" COPYONLY)
+endif()
+
+if(NOT HEAD_HASH)
+	file(READ "@GIT_DATA@/head-ref" HEAD_HASH LIMIT 1024)
+	string(STRIP "${HEAD_HASH}" HEAD_HASH)
+endif()

cmake/modules/NSIS.template.in

@@ -6,6 +6,7 @@
 !define APPLICATION_EXECUTABLE "@APPLICATION_EXECUTABLE@.exe"
 !define APPLICATION_DOMAIN "@APPLICATION_DOMAIN@"
 !define APPLICATION_LICENSE "@APPLICATION_LICENSE@"
+!define WIN_SETUP_BITMAP_PATH "@WIN_SETUP_BITMAP_PATH@"
 
 !define MUI_FINISHPAGE_LINK_LOCATION "http://www.${APPLICATION_DOMAIN}"
 
@@ -103,11 +104,11 @@ ReserveFile "${NSISDIR}\Plugins\InstallOptions.dll"
 !define MUI_ABORTWARNING
 !define MUI_ICON ${NSI_PATH}\installer.ico
 !define MUI_UNICON ${NSI_PATH}\installer.ico
-!define MUI_WELCOMEFINISHPAGE_BITMAP ${NSI_PATH}\welcome_${APPLICATION_SHORTNAME}.bmp
+!define MUI_WELCOMEFINISHPAGE_BITMAP ${WIN_SETUP_BITMAP_PATH}/welcome.bmp
 !define MUI_WELCOMEPAGE_TITLE "Welcome to the @CPACK_PACKAGE_NAME@ ${VERSION} Setup Wizard"
 !define MUI_WELCOMEPAGE_TEXT "This wizard will guide you through the installation.$\r$\n$\r$\n$_CLICK"
 !define MUI_HEADERIMAGE
-!define MUI_HEADERIMAGE_BITMAP ${NSI_PATH}\page_header_${APPLICATION_SHORTNAME}.bmp
+!define MUI_HEADERIMAGE_BITMAP  ${WIN_SETUP_BITMAP_PATH}/page_header.bmp
 !define MUI_COMPONENTSPAGE_SMALLDESC
 !define MUI_FINISHPAGE_TITLE "@CPACK_PACKAGE_NAME@ Setup Completed"
 !define MUI_FINISHPAGE_LINK "Click here to visit the @CPACK_PACKAGE_NAME@ website."
@@ -128,7 +129,7 @@ ReserveFile "${NSISDIR}\Plugins\InstallOptions.dll"
 ;-----------------------------------------------------------------------------
 !insertmacro MUI_PAGE_WELCOME
 !ifdef OPTION_LICENSE_AGREEMENT
-   !insertmacro MUI_PAGE_LICENSE "${source_path}/${APPLICATION_LICENSE}"
+   !insertmacro MUI_PAGE_LICENSE "${APPLICATION_LICENSE}"
 !endif
 Page custom PageReinstall PageLeaveReinstall
 !insertmacro MUI_PAGE_COMPONENTS
@@ -324,6 +325,9 @@ Section "${APPLICATION_NAME}" SEC_OWNCLOUD
    File "${QT_DLL_PATH}\QtNetwork4.dll"
    File "${QT_DLL_PATH}\QtXml4.dll"
 
+   ;QtKeyChain stuff
+   File "${MING_BIN}\libqtkeychain.dll"
+
    File "${CSYNC_LIBRARY_DIR}/libocsync.dll"
    File "${MING_BIN}\libsqlite3-0.dll"
    File "${MING_BIN}\libiniparser.dll"
@@ -331,14 +335,10 @@ Section "${APPLICATION_NAME}" SEC_OWNCLOUD
    File "${MING_BIN}\libpng15-15.dll"
 
    ; ownCloud plugin
-   File "${MING_BIN}\libgcrypt-11.dll"
-   File "${MING_BIN}\libgnutls-26.dll"
-   File "${MING_BIN}\libgpg-error-0.dll"
    File "${MING_BIN}\libintl-8.dll"
    File "${MING_BIN}\libneon-27.dll"
    File "${MING_BIN}\libproxy.dll"
    File "${MING_BIN}\libmodman.dll"
-   File "${MING_BIN}\libtasn1-3.dll"
    File "${MING_BIN}\libxml2-2.dll"
 
    ;MinGW stuff

config.h.in

@@ -3,6 +3,10 @@
 
 #cmakedefine USE_INOTIFY 1
 #cmakedefine WITH_CSYNC 1
+#cmakedefine WITH_QTKEYCHAIN 1
 
+#cmakedefine GIT_SHA1 "@GIT_SHA1@"
+#cmakedefine APPLICATION_DOMAIN @APPLICATION_DOMAIN@
 #cmakedefine THEME_CLASS @THEME_CLASS@
+#cmakedefine THEME_INCLUDE @THEME_INCLUDE@
 #endif

doc/CMakeLists.txt

@@ -9,6 +9,8 @@ if(SPHINX_FOUND)
     set(SPHINX_QCH_DIR      "${CMAKE_CURRENT_BINARY_DIR}/qthelp")
     set(SPHINX_HTMLHELP_DIR "${CMAKE_CURRENT_BINARY_DIR}/htmlhelp")
     set(MSHTML_COMPILER     wine 'C:\\Program Files\\HTML Help Workshop\\hhc.exe')
+    # assets
+    set(LATEX_LOGO          "${CMAKE_CURRENT_SOURCE_DIR}/logo-blue.pdf")
 
     configure_file("${CMAKE_CURRENT_SOURCE_DIR}/conf.py.in" conf.py @ONLY)
     add_custom_target(doc DEPENDS doc-html doc-man COMMENT "Building documentation...")
@@ -18,6 +20,7 @@ if(SPHINX_FOUND)
         add_custom_target(doc-latex  ${SPHINX_EXECUTABLE}
 		                     -q -c . -b latex
 				     -d ${SPHINX_CACHE_DIR}
+				     -D latex_logo=${LATEX_LOGO}
 				     ${CMAKE_CURRENT_SOURCE_DIR}
 				     ${SPHINX_PDF_DIR} )
         add_custom_target(doc-pdf make -C ${SPHINX_PDF_DIR} all-pdf

doc/branding.rst

@@ -1,87 +0,0 @@
-Adding 3rd-Party Branding
-=========================
-
-The ownCloud Client supports theming in the setup assistant dialog in both the
-account setting and the result window. These customizations do not require a
-recompile of the client.
-
-The dialogs have places where custom content can be inserted. Please take a look
-on the screenshot below. For example the connection wizard has the two green
-rectangles as well as the area at the bottom with the black line and the red
-text.
-
-The content places can be altered by a file called ``custom.ini`` which contains
-content for the specific places in the dialogs.
-
-On windows, the custom.ini has to be stored at
-
-+-----------------+------------------------------------------------------------+
-| OS              | Path                                                       |
-+=================+============================================================+
-| Windows (64 bit)| ``%ProgramFiles(x86)%\ownCloud\custom.ini``                |
-+-----------------+------------------------------------------------------------+
-| Windows (32 bit)| ``%ProgramFiles%\ownCloud\custom.ini``                     |
-+-----------------+------------------------------------------------------------+
-| Mac OS          | ``owncloud.app/Contents/Resources/custom.ini``             |
-+-----------------+------------------------------------------------------------+
-| Linux           | ``/etc/ownCloud/custom.ini``                               |
-+-----------------+------------------------------------------------------------+
-
-The config file needs to be in a usual `Windows INI file format`_.
-
-Under the group definition [GUICustomize] the following keys can be set:
-
-* ``oCSetupTop``: The top area of the connect wizard, account page.
-* ``oCSetupSide``: The side area of the connect wizard, account page.
-* ``ocSetupBottom``: The bottom area of the connect wizard, account page
-* ``oCSetupResultTop``: The top area of the connect wizard result page.
-* ``oCSetupFixUrl``: A fixed url to the ownCloud instance. If this is set, the
-  user can not access the url entry field and it is hidden.
-
-The keys can be set to either an image file or a `rich text`_ snippet, which
-must be enclosed with  double quotes. Image files must be addressed by the
-complete file name. On windows, the pathes should be noted with slashes
-instead of backslashes. Instead of absolute file pathes, you can use
-the ``%RESOURCES`` placeholder (note the single ``%``!).
-
-Example::
-
-  %RESOURCES/custom/myimage_top.png
-
-.. note:: ``%RESOURCES`` is currently undefined on Linux, it points to the
-          directory location of ``custom.ini`` on all other platforms.
-
-Example
-~~~~~~~
-
-A config file ``custom.ini`` like this::
-
-  [GUICustomize]
-  oCSetupTop=/home/kf/owncloud.com/topimg.png
-  oCSetupSide=/home/kf/owncloud.com/sideimg.png
-  oCSetupBottom="
-  <hr />
-  The Power of now: ownCloud Client
-
-  This nice ownCloud Client was presented by <a
-  href="http://owncloud.com">ownCloud Inc.</a>
-  "
-  oCSetupResultTop=/home/kf/owncloud.com/topimg.png
-
-will produce this setup dialog:
-
-.. figure:: images/branding_setup.png
-
-  Setup Dialog Customization
-
-Packaging remarks
-~~~~~~~~~~~~~~~~~
-
-While you do not need to rebuild the client, you will need to repackage
-the client on your own. This also means you will have to resign those
-packages with your own signing key.
-
-.. _`Windows INI file format`: http://en.wikipedia.org/wiki/INI_file
-.. _`rich text`: http://qt-project.org/doc/qt-5.0/richtext-html-subset.html 
-
-

doc/building.rst

@@ -21,7 +21,7 @@ Mac OS X
 --------
 
 Follow the `generic build instructions`_.
- 
+
 You can install the missing dependencies via MacPorts_ or Homebrew_.
 This is only needed on the build machine, since non-standard libs
 will be deployed in the app bundle.
@@ -33,7 +33,7 @@ build system. If you are using Homebrew_, you can just add it::
   brew install iniparser
 
 Otherwise, you need to copy the header and lib files to
-``/usr/local/include`` and ``/usr/local/lib`` respectively. 
+``/usr/local/include`` and ``/usr/local/lib`` respectively.
 
 .. note::
   You should not call ``make install`` at any time, since the product of the
@@ -51,22 +51,33 @@ if you do not have it installed already.
 In order to cross-compile, the following repositories need to be added
 via YaST or ``zypper ar`` (adjust when using openSUSE 12.2)::
 
-  http://download.opensuse.org/repositories/isv:/ownCloud:/devel:/mingw:/win32/openSUSE_12.1/isv:ownCloud:devel:mingw:win32.repo
-  http://download.opensuse.org/repositories/windows:/mingw/openSUSE_12.1/windows:mingw.repo
-  http://download.opensuse.org/repositories/windows:/mingw:/win32/openSUSE_12.1/windows:mingw:win32.repo
+  http://download.opensuse.org/repositories/isv:/ownCloud:/devel:/mingw:/win32/openSUSE_12.1/
+  http://download.opensuse.org/repositories/windows:/mingw/openSUSE_12.1/
+  http://download.opensuse.org/repositories/windows:/mingw:/win32/openSUSE_12.1/
 
 Next, install the cross-compiler packages and the cross-compiled dependencies::
 
   zypper si -d mingw32-csync
-  zypper install kdewin-png2ico mingw32-libqt4 mingw32-libqt4-devel
-                 mingw32-libgcrypt mingw32-libgnutls mingw32-gnutls \
-                 mingw32-libneon mingw32-libbeecrypt mingw32-libopenssl \
-                 mingw32-openssl
+  zypper install kdewin-png2ico mingw32-libqt4 mingw32-libqt4-devel \
+                 mingw32-libgcrypt mingw32-libgnutls mingw32-libneon \
+                 mingw32-libbeecrypt mingw32-libopenssl mingw32-openssl \
+                 mingw32-libpng-devel mingw32-libsqlite mingw32-qtkeychain \
+                 mingw32-qtkeychain-devel mingw32-iniparser mingw32-dlfcn
 
-For the installer, the NSIS installer packages are also required::
+For the installer, the NSIS installer package is also required::
 
-  zypper install mingw32-cross-nsis mingw32-cross-nsis-plugin-processes \
-                 mingw32-cross-nsis-plugin-uac 
+  zypper install mingw32-cross-nsis
+
+..  Usually, the following would be needed as well, but due to a bug in mingw, they
+    will currently not build properly from source.
+
+    mingw32-cross-nsis-plugin-processes mingw32-cross-nsis-plugin-uac
+
+You will also need to manually download and install the following files with
+``rpm -ivh <package>`` (They will also work with OpenSUSE 12.2)::
+
+http://pmbs.links2linux.org/download/mingw:/32/openSUSE_12.1/x86_64/mingw32-cross-nsis-plugin-processes-0-1.1.x86_64.rpm
+http://pmbs.links2linux.org/download/mingw:/32/openSUSE_12.1/x86_64/mingw32-cross-nsis-plugin-uac-0-3.1.x86_64.rpm
 
 Now, follow the `generic build instructions`_, but pay attention to
 the following differences:
@@ -111,27 +122,29 @@ Next, make sure to check out the 'dav' branch in the newly checked out
   cd ocsync
   git checkout dav
 
-The first package to buidld is CSync::
+The first package to build is CSync::
 
   cd ocsync-build
-  cmake -DCMAKE_BUILD_TYPE="Debug" -DLOG_TO_CALLBACK=ON -DWITH_LOG4C=OFF ../ocsync
+  cmake -DCMAKE_BUILD_TYPE="Debug" ../ocsync
   make
 
 You probably have to satisfy some dependencies. Make sure to install all the
 needed development packages. You will need ``iniparser``, ``sqlite3`` as well as
-``neon`` for the ownCloud module. ``libssh`` and ``libsmbclient`` are optional
-and not required for the client to work. If you want to install the client, run
-``make install`` as a final step.
+``neon`` for the ownCloud module. Take special care about ``neon``. If that is
+missing, the cmake run will succeed but silently not build the ownCloud module.
+``libssh`` and ``libsmbclient`` are optional and not required for the client
+to work. If you want to install the client, run ``make install`` as a final step.
 
 Next, we build mirall::
 
   cd ../mirall-build
   cmake -DCMAKE_BUILD_TYPE="Debug" ../mirall \
-        -DCSYNC_LIBRARY_PATH=../ocsync-build \
-        -DCSYNC_INCLUDE_PATH=../ocsync/src
+        -DCSYNC_BUILD_PATH=/path/to/ocsync-build \
+        -DCSYNC_INCLUDE_PATH=/path/to/ocsync/src
 
-If this succeeds, call ``make``. The owncloud binary should appear in the
-``bin`` directory. You can also run ``make install`` to install the client to
+Note that it is important to use absolute pathes for the include- and library
+directories. If this succeeds, call ``make``. The owncloud binary should appear
+in the ``bin`` directory. You can also run ``make install`` to install the client to
 ``/usr/local/bin``.
 
 To build in installer (requires the mingw32-cross-nsis packages)::

doc/index.rst

@@ -1,10 +1,4 @@
-.. ownCloud Documentation documentation master file, created by
-   sphinx-quickstart on Mon Oct 22 23:16:40 2012.
-   You can adapt this file completely to your liking, but it should at least
-   contain the root `toctree` directive.
-
-ownCloud Desktop Client
-***********************
+.. _contents:
 
 Contents
 ========
@@ -15,7 +9,6 @@ Contents
    install
    usage 
    building 
-   branding
    architecture
    troubleshooting
    glossary 

doc/usage.rst

@@ -36,7 +36,8 @@ Config File
 .. index:: config file
 
 ownCloud Client reads a configuration file which on Linux can be found at ``$HOME/.local/share/data/ownCloud/owncloud.cfg``
-.. todo:: Windows, Mac?
+On Windows, it can be found in ``\Users\<name>\AppData\Local\ownCloud\owncloud.cfg``
+.. todo:: Mac?
 It contains settings in the ini file format known from Windows. 
 
 .. note:: Changes here should be done carefully as wrong settings can cause disfunctionality.
@@ -49,18 +50,9 @@ These are config settings that may be changed:
 +===========================+===========+==============+===========+=====================================================+
 | ``remotePollinterval``    | integer   | milliseconds | ``30000`` | Poll time for the remote repository                 |
 +---------------------------+-----------+--------------+-----------+-----------------------------------------------------+
-| ``localPollinterval``     | integer   | milliseconds | ``10000`` | Poll time for local repository                      |
-+---------------------------+-----------+--------------+-----------+-----------------------------------------------------+
-| ``PollTimerExceedFactor`` | integer   | n/a          | ``10``    | Poll Timer Exceed Factor                            |
-+---------------------------+-----------+--------------+-----------+-----------------------------------------------------+
 | ``maxLogLines``           | integer   | lines        | ``20000`` | Maximum count of log lines shown in the log window  |
 +---------------------------+-----------+--------------+-----------+-----------------------------------------------------+
 
-* ``remotePollinterval`` is for systems which have notify for local file system changes (Linux only currently)
-  this is the frequency it polls for remote changes. The following two values are ignored.
+* ``remotePollinterval`` The frequency used for polling for remote changes on
+  the ownCloud Server.
 
-* ``localPollinterval`` is for systems which poll the local file system (currently Win and Mac) this is the
-  frequency they poll locally. The ``remotePollInterval`` is ignored on these systems.
-
-* ``PollTimerExceedFactor`` sets  the exceed factor is the factor after which a remote poll is done. That means the effective
-  frequency for remote poll is ``localPollInterval * pollTimerExceedFactor``.

issue_template.md

@@ -0,0 +1,48 @@
+### Expected behaviour
+Tell us what should happen
+
+### Actual behaviour
+Tell us what happens instead
+
+### Steps to reproduce
+1. 
+2. 
+3. 
+
+### Server configuration
+Operating system:
+
+Web server: 
+
+Database:
+
+PHP version:
+
+ownCloud version:
+
+### Client configuration
+Client version:
+
+Operating system:
+
+OS language:
+
+Installation path of client:
+
+### Logs
+
+#### output of `owncloud --logwindow` or `owncloud --logfile log.txt`
+```
+Insert your log output here
+```
+
+#### Web server error log
+```
+Insert your webserver log here
+```
+
+#### ownCloud log (data/owncloud.log)
+```
+Insert your ownCloud log here
+```
+

mirall.desktop.in

@@ -0,0 +1,7 @@
+[Desktop Entry]
+Categories=Utility;X-SuSE-SyncUtility;
+Type=Application
+Exec=@APPLICATION_EXECUTABLE@
+Name=@APPLICATION_NAME@ desktop sync client 
+GenericName=Folder Sync
+Icon=@APPLICATION_SHORTNAME@

mirall.qrc

@@ -19,19 +19,6 @@
         <file>resources/folder-sync-48.png</file>
         <file>resources/folder-important.png</file>
         <file>resources/folder-grey.png</file>
-        <file>resources/owncloud_splash.png</file>
         <file>resources/task-ongoing.png</file>
-
-        <file>resources/owncloud-icon-22.png</file>
-        <file>resources/owncloud-icon-32.png</file>
-        <file>resources/owncloud-icon-48.png</file>
-        <file>resources/owncloud-icon-64.png</file>
-        <file>resources/owncloud-icon-128.png</file>
-        <file>resources/owncloud-framed-64.png</file>
-        <file>resources/owncloud-error-48.png</file>
-        <file>resources/owncloud-sync-48.png</file>
-        <file>resources/owncloud-sync-ok-48.png</file>
-
-        <file>resources/warning-16.png</file>
     </qresource>
 </RCC>

src/CMakeLists.txt

@@ -3,6 +3,13 @@ include(${QT_USE_FILE})
 include_directories(${CMAKE_CURRENT_SOURCE_DIR} ${CMAKE_CURRENT_BINARY_DIR})
 
 QT4_ADD_RESOURCES ( MIRALL_RC_SRC ../mirall.qrc)
+if ( DEFINED OEM_THEME_DIR)
+    QT4_ADD_RESOURCES ( MIRALL_RC_SRC ${OEM_THEME_DIR}/theme.qrc)
+    set(theme_dir ${OEM_THEME_DIR}/theme)
+else()
+    QT4_ADD_RESOURCES ( MIRALL_RC_SRC ../theme.qrc)
+    set(theme_dir ${CMAKE_CURRENT_SOURCE_DIR}/../theme)
+endif()
 
 set(mirall_UI
 mirall/folderwizardsourcepage.ui
@@ -18,6 +25,7 @@ mirall/owncloudwizardresultpage.ui
 mirall/owncloudcredentialspage.ui
 mirall/sslerrordialog.ui
 mirall/proxydialog.ui
+mirall/fileitemdialog.ui
 )
 
 set(3rdparty_SRC
@@ -70,6 +78,8 @@ set(libsync_SRCS
     mirall/owncloudtheme.cpp
     mirall/miralltheme.cpp
     mirall/owncloudinfo.cpp
+    mirall/logger.cpp
+    mirall/utility.cpp
 )
 set(libsync_HEADERS 
     mirall/folderman.h
@@ -82,20 +92,47 @@ set(libsync_HEADERS
     mirall/csyncthread.h
     mirall/owncloudinfo.h
     mirall/credentialstore.h
+    mirall/logger.h
 )
 
 IF( INOTIFY_FOUND )
     set(libsync_SRCS ${libsync_SRCS} mirall/inotify.cpp)
+    set(libsync_SRCS ${libsync_SRCS} mirall/folderwatcher_inotify.cpp)
     set(libsync_HEADERS ${libsync_HEADERS} mirall/inotify.h)
+    set(libsync_HEADERS ${libsync_HEADERS} mirall/folderwatcher_inotify.h)
+ENDIF()
+IF( WIN32 ) 
+    set(libsync_SRCS ${libsync_SRCS} mirall/folderwatcher_win.cpp)
+    set(libsync_HEADERS ${libsync_HEADERS} mirall/folderwatcher_win.h)
+ENDIF()
+IF( APPLE )
+    set(libsync_SRCS ${libsync_SRCS} mirall/folderwatcher_mac.cpp)
 ENDIF()
 
+
 qt4_wrap_cpp(syncMoc ${libsync_HEADERS})
 
+list(APPEND libsync_LINK_TARGETS
+    ${QT_LIBRARIES}
+    ${CSYNC_LIBRARY}
+)
+
+if(QTKEYCHAIN_FOUND)
+    list(APPEND libsync_LINK_TARGETS ${QTKEYCHAIN_LIBRARY})
+    include_directories(${QTKEYCHAIN_INCLUDE_DIR})
+endif()
+
 add_library(mirallsync   SHARED ${libsync_SRCS} ${syncMoc})
 add_library(owncloudsync SHARED ${libsync_SRCS} ${syncMoc})
 set_target_properties( owncloudsync PROPERTIES COMPILE_DEFINITIONS OWNCLOUD_CLIENT)
-target_link_libraries(mirallsync ${QT_LIBRARIES} ${CSYNC_LIBRARY} )
-target_link_libraries(owncloudsync ${QT_LIBRARIES} ${CSYNC_LIBRARY} )
+
+target_link_libraries(mirallsync ${libsync_LINK_TARGETS} )
+target_link_libraries(owncloudsync ${libsync_LINK_TARGETS} )
+
+if ( APPLE )
+    target_link_libraries(mirallsync /System/Library/Frameworks/CoreServices.framework)
+    target_link_libraries(owncloudsync /System/Library/Frameworks/CoreServices.framework)
+endif()
 
 if(NOT BUILD_OWNCLOUD_OSX_BUNDLE)
     install(TARGETS mirallsync owncloudsync
@@ -103,6 +140,11 @@ if(NOT BUILD_OWNCLOUD_OSX_BUNDLE)
         LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
         ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
     )
+    if(NOT WIN32)
+        configure_file(${CMAKE_SOURCE_DIR}/mirall.desktop.in
+                       ${APPLICATION_SHORTNAME}.desktop)
+        install(FILES  ${CMAKE_CURRENT_BINARY_DIR}/${APPLICATION_SHORTNAME}.desktop DESTINATION share/applications )
+    endif()
 else()
     install(TARGETS mirallsync owncloudsync DESTINATION ${OWNCLOUD_OSX_BUNDLE}/Contents/MacOS)
 endif()
@@ -118,6 +160,7 @@ set(mirall_SRCS
     mirall/sslerrordialog.cpp
     mirall/logbrowser.cpp
     mirall/proxydialog.cpp
+    mirall/fileitemdialog.cpp
 )
 
 set(mirall_HEADERS
@@ -130,6 +173,7 @@ set(mirall_HEADERS
     mirall/sslerrordialog.h
     mirall/logbrowser.h
     mirall/proxydialog.h
+    mirall/fileitemdialog.h
 )
 
 if( UNIX AND NOT APPLE)
@@ -161,15 +205,21 @@ set( final_src
 # add executable icon on windows and osx
 include( AddAppIconMacro )
 set(ownCloud_old ${ownCloud})
-kde4_add_app_icon( ownCloud "${CMAKE_CURRENT_SOURCE_DIR}/../resources/owncloud-icon-*.png") 
+
+kde4_add_app_icon( ownCloud "${theme_dir}/colored/${APPLICATION_SHORTNAME}-icon*.png")
 list(APPEND final_src ${ownCloud})
 set(ownCloud ${ownCloud_old})
 
 if(NOT BUILD_OWNCLOUD_OSX_BUNDLE)
     set(BIN_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}/bin")
 
-    install(FILES ${CMAKE_SOURCE_DIR}/resources/mirall-48.png DESTINATION share/icons/hicolor/48x48/apps/ RENAME mirall.png)
-    install(FILES ${CMAKE_SOURCE_DIR}/resources/owncloud-icon-48.png DESTINATION share/icons/hicolor/48x48/apps/ RENAME owncloud.png)
+if(NOT WIN32)
+    install(FILES ${CMAKE_SOURCE_DIR}/resources/mirall-48.png 
+        DESTINATION share/icons/hicolor/48x48/apps/ RENAME mirall.png)
+    install(FILES
+        ${theme_dir}/colored/${APPLICATION_SHORTNAME}-icon-48.png 
+        DESTINATION share/icons/hicolor/48x48/apps/ RENAME ${APPLICATION_SHORTNAME}.png)
+endif(NOT WIN32)
 
     install(FILES ${mirall_I18N} DESTINATION share/mirall/i18n)
 

src/mirall/application.cpp

@@ -12,7 +12,6 @@
  * for more details.
  */
 
-#define LOG_TO_CALLBACK // FIXME: This should be in csync.
 #include <iostream>
 
 #include "mirall/application.h"
@@ -32,14 +31,17 @@
 #include "mirall/proxydialog.h"
 #include "mirall/version.h"
 #include "mirall/credentialstore.h"
+#include "mirall/logger.h"
+
+// for version information
+#include "config.h"
+#include <csync.h>
 
 #ifdef WITH_CSYNC
 #include "mirall/csyncfolder.h"
 #endif
 #include "mirall/inotify.h"
 
-#include <csync.h>
-
 #include <QtCore>
 #include <QtGui>
 #include <QHash>
@@ -59,27 +61,23 @@ void mirallLogCatcher(QtMsgType type, const char *msg)
   Logger::instance()->mirallLog( QString::fromUtf8(msg) );
 }
 
-void csyncLogCatcher(const char *msg)
-{
-  Logger::instance()->csyncLog( QString::fromUtf8(msg) );
-}
-
 // ----------------------------------------------------------------------------------
 
 Application::Application(int &argc, char **argv) :
     SharedTools::QtSingleApplication(argc, argv),
     _tray(0),
-    _sslErrorDialog(0),
 #if QT_VERSION >= 0x040700
     _networkMgr(new QNetworkConfigurationManager(this)),
 #endif
+    _sslErrorDialog(0),
     _contextMenu(0),
     _theme(Theme::instance()),
     _updateDetector(0),
+    _logBrowser(0),
     _showLogWindow(false),
     _logFlush(false),
     _helpOnly(false),
-    _logBrowser(0)
+    _fileItemDialog(0)
 {
     setApplicationName( _theme->appName() );
     setWindowIcon( _theme->applicationIcon() );
@@ -88,7 +86,6 @@ Application::Application(int &argc, char **argv) :
     setupLogBrowser();
     //no need to waste time;
     if ( _helpOnly ) return;
-    processEvents();
 
     QTranslator *qtTranslator = new QTranslator(this);
     qtTranslator->load(QLatin1String("qt_") + QLocale::system().name(),
@@ -114,7 +111,8 @@ Application::Application(int &argc, char **argv) :
     installTranslator(mirallTranslator);
 
     connect( this, SIGNAL(messageReceived(QString)), SLOT(slotParseOptions(QString)));
-
+    connect( Logger::instance(), SIGNAL(guiLog(QString,QString)),
+             this, SLOT(slotShowTrayMessage(QString,QString)));
     // create folder manager for sync folder management
     _folderMan = new FolderMan(this);
     connect( _folderMan, SIGNAL(folderSyncStateChange(QString)),
@@ -130,7 +128,8 @@ Application::Application(int &argc, char **argv) :
     _folderWizard = new FolderWizard;
 
     _owncloudSetupWizard = new OwncloudSetupWizard( _folderMan, _theme, this );
-    connect( _owncloudSetupWizard, SIGNAL(ownCloudWizardDone(int)), SLOT(slotStartFolderSetup(int)));
+    connect( _owncloudSetupWizard, SIGNAL(ownCloudWizardDone(int)),
+             this, SLOT(slotownCloudWizardDone(int)));
 
     _statusDialog = new StatusDialog( _theme );
     connect( _statusDialog, SIGNAL(addASync()), this, SLOT(slotAddFolder()) );
@@ -147,17 +146,17 @@ Application::Application(int &argc, char **argv) :
     connect( _statusDialog, SIGNAL(openFolderAlias(const QString&)),
              SLOT(slotFolderOpenAction(QString)));
 
+#if 0
 #if QT_VERSION >= 0x040700
     qDebug() << "* Network is" << (_networkMgr->isOnline() ? "online" : "offline");
     foreach (const QNetworkConfiguration& netCfg, _networkMgr->allConfigurations(QNetworkConfiguration::Active)) {
         //qDebug() << "Network:" << netCfg.identifier();
     }
 #endif
-
+#endif
     setupActions();
     setupSystemTray();
     setupProxy();
-    processEvents();
 
     QObject::connect( this, SIGNAL(messageReceived(QString)),
                          this, SLOT(slotOpenStatus()) );
@@ -178,6 +177,8 @@ Application::Application(int &argc, char **argv) :
 Application::~Application()
 {
     delete _tray; // needed, see ctor
+    if( _fileItemDialog) delete _fileItemDialog;
+    if( _statusDialog )  delete _statusDialog;
     qDebug() << "* Mirall shutdown";
 }
 
@@ -260,11 +261,22 @@ void Application::slotNoOwnCloudFound( QNetworkReply* reply )
 
 void Application::slotFetchCredentials()
 {
-    connect( CredentialStore::instance(), SIGNAL(fetchCredentialsFinished(bool)),
-             this, SLOT(slotCredentialsFetched(bool)) );
-    CredentialStore::instance()->fetchCredentials();
-    if( CredentialStore::instance()->state() == CredentialStore::TooManyAttempts ) {
-        QString trayMessage = tr("Too many user attempts to enter password.");
+    QString trayMessage;
+
+    if( CredentialStore::instance()->canTryAgain() ) {
+        connect( CredentialStore::instance(), SIGNAL(fetchCredentialsFinished(bool)),
+                 this, SLOT(slotCredentialsFetched(bool)) );
+        CredentialStore::instance()->fetchCredentials();
+        if( CredentialStore::instance()->state() == CredentialStore::TooManyAttempts ) {
+            trayMessage = tr("Too many attempts to get a valid password.");
+        }
+    } else {
+        qDebug() << "Can not try again to fetch Credentials.";
+        trayMessage = tr("%1 user credentials are wrong. Please check configuration.")
+                .arg(Theme::instance()->appName());
+    }
+
+    if( !trayMessage.isEmpty() ) {
         _tray->showMessage(tr("Credentials"), trayMessage);
         _actionOpenStatus->setEnabled( false );
         _actionAddFolder->setEnabled( false );
@@ -279,6 +291,8 @@ void Application::slotCredentialsFetched(bool ok)
         trayMessage = tr("Error: Could not retrieve the password!");
         if( CredentialStore::instance()->state() == CredentialStore::UserCanceled ) {
             trayMessage = tr("Password dialog was canceled!");
+        } else {
+            trayMessage = CredentialStore::instance()->errorMessage();
         }
 
         if( !trayMessage.isEmpty() ) {
@@ -336,9 +350,8 @@ void Application::slotAuthCheck( const QString& ,QNetworkReply *reply )
         qDebug() << "######## Credentials are ok!";
         int cnt = _folderMan->setupFolders();
         if( cnt ) {
-            _tray->setIcon(_theme->applicationIcon());
+            _tray->setIcon( _theme->syncStateIcon( SyncResult::NotYetStarted, true ) );
             _tray->show();
-            processEvents();
 
             if( _tray )
                 _tray->showMessage(tr("%1 Sync Started").arg(_theme->appName()),
@@ -346,8 +359,10 @@ void Application::slotAuthCheck( const QString& ,QNetworkReply *reply )
 
             _statusDialog->setFolderList( _folderMan->map() );
             computeOverallSyncStatus();
+
         }
         _actionAddFolder->setEnabled( true );
+        _actionOpenStatus->setEnabled( true );
         setupContextMenu();
     } else {
         slotFetchCredentials();
@@ -390,12 +405,21 @@ void Application::slotSSLFailed( QNetworkReply *reply, QList<QSslError> errors )
     }
 }
 
+void Application::slotownCloudWizardDone( int res )
+{
+    if( res == QDialog::Accepted ) {
+
+    }
+    slotStartFolderSetup( res );
+}
+
 void Application::setupActions()
 {
     _actionOpenoC = new QAction(tr("Open %1 in browser...").arg(_theme->appName()), this);
     QObject::connect(_actionOpenoC, SIGNAL(triggered(bool)), SLOT(slotOpenOwnCloud()));
     _actionOpenStatus = new QAction(tr("Open status..."), this);
     QObject::connect(_actionOpenStatus, SIGNAL(triggered(bool)), SLOT(slotOpenStatus()));
+    _actionOpenStatus->setEnabled( false );
     _actionAddFolder = new QAction(tr("Add folder..."), this);
     QObject::connect(_actionAddFolder, SIGNAL(triggered(bool)), SLOT(slotAddFolder()));
     _actionConfigure = new QAction(tr("Configure..."), this);
@@ -413,7 +437,7 @@ void Application::setupSystemTray()
     // Setting a parent heres will crash on X11 since by the time qapp runs
     // its childrens dtors, the X11->screen variable queried for is gone -> crash
     _tray = new QSystemTrayIcon;
-    _tray->setIcon( _theme->applicationIcon() ); // load the grey icon
+    _tray->setIcon( _theme->syncStateIcon( SyncResult::NotYetStarted, true ) );
 
     connect(_tray,SIGNAL(activated(QSystemTrayIcon::ActivationReason)),
             SLOT(slotTrayClicked(QSystemTrayIcon::ActivationReason)));
@@ -489,8 +513,11 @@ void Application::setupContextMenu()
     _contextMenu->addAction(_actionConfigure);
     _contextMenu->addAction(_actionConfigureProxy);
     _contextMenu->addSeparator();
-    _contextMenu->addAction(_actionAbout);
-    _contextMenu->addSeparator();
+
+    if (!Theme::instance()->about().isEmpty()) {
+        _contextMenu->addAction(_actionAbout);
+        _contextMenu->addSeparator();
+    }
 
     _contextMenu->addAction(_actionQuit);
 }
@@ -502,7 +529,6 @@ void Application::setupLogBrowser()
         // init the log browser.
         _logBrowser = new LogBrowser;
         qInstallMsgHandler( mirallLogCatcher );
-        csync_set_log_callback( csyncLogCatcher );
         // ## TODO: allow new log name maybe?
         if (!_logFile.isEmpty()) {
             qDebug() << "Logging into logfile: " << _logFile << " with flush " << _logFlush;
@@ -590,7 +616,8 @@ void Application::slotTrayClicked( QSystemTrayIcon::ActivationReason reason )
     // A click on the tray icon should only open the status window on Win and
     // Linux, not on Mac. They want a menu entry.
     // If the user canceled login, rather open the login window.
-    if( CredentialStore::instance()->state() == CredentialStore::UserCanceled ) {
+    if( CredentialStore::instance()->state() == CredentialStore::UserCanceled ||
+            CredentialStore::instance()->state() == CredentialStore::Error ) {
         slotFetchCredentials();
     }
 #if defined Q_WS_WIN || defined Q_WS_X11
@@ -630,6 +657,8 @@ void Application::slotAddFolder()
         targetPath  = _folderWizard->field(QLatin1String("targetURLFolder")).toString();
         onlyOnline  = _folderWizard->field(QLatin1String("onlyOnline?")).toBool();
         onlyThisLAN = _folderWizard->field(QLatin1String("onlyThisLAN?")).toBool();
+        (void) onlyOnline;
+        (void) onlyThisLAN;
     } else if( _folderWizard->field(QLatin1String("OC?")).toBool() ||
                Theme::instance()->singleSyncFolder()) {
         // setup a ownCloud folder
@@ -680,7 +709,11 @@ void Application::slotOpenStatus()
       _statusDialog->setFolderList( _folderMan->map() );
     }
   }
+  raiseDialog( raiseWidget );
+}
 
+void Application::raiseDialog( QWidget *raiseWidget )
+{
   // viel hilft viel ;-)
   if( raiseWidget ) {
 #if defined(Q_WS_WIN) || defined (Q_OS_MAC)
@@ -705,10 +738,20 @@ void Application::slotOpenLogBrowser()
 
 void Application::slotAbout()
 {
+    QString devString;
+#ifdef GIT_SHA1
+    const QString githubPrefix(QLatin1String(
+                               "https://github.com/owncloud/mirall/commit/"));
+    const QString gitSha1(QLatin1String(GIT_SHA1));
+    devString = tr("<p><small>Built from Git revision <a href=\"%1\">%2</a>"
+                   " on %3, %4<br>using OCsync %5 and Qt %6.</small><p>")
+                       .arg(githubPrefix+gitSha1).arg(gitSha1.left(6))
+                       .arg(__DATE__).arg(__TIME__)
+                       .arg(MIRALL_STRINGIFY(LIBCSYNC_VERSION))
+                       .arg(QT_VERSION_STR);
+#endif
     QMessageBox::about(0, tr("About %1").arg(_theme->appName()),
-                       tr("%1 client, version %2\n\nCopyright 2012, the ownCloud developers.")
-                       .arg(_theme->appName())
-                       .arg(MIRALL_STRINGIFY(MIRALL_VERSION)));
+                       Theme::instance()->about());
 }
 
 /*
@@ -735,80 +778,19 @@ void Application::slotRemoveFolder( const QString& alias )
     setupContextMenu();
 }
 
+// Open the File list info dialog.
 void Application::slotInfoFolder( const QString& alias )
 {
     qDebug() << "details of folder with alias " << alias;
 
+    if( !_fileItemDialog ) {
+        _fileItemDialog = new FileItemDialog(_theme);
+    }
+
     SyncResult folderResult = _folderMan->syncResult( alias );
 
-    bool enabled = true;
-    Folder *f = _folderMan->folder( alias );
-    if( f && ! f->syncEnabled() ) {
-        enabled = false;
-    }
-
-    QString folderMessage;
-
-    SyncResult::Status syncStatus = folderResult.status();
-    switch( syncStatus ) {
-    case SyncResult::Undefined:
-        folderMessage = tr( "Undefined Folder State" );
-        break;
-    case SyncResult::NotYetStarted:
-        folderMessage = tr( "The folder waits to start syncing." );
-        break;
-    case SyncResult::SyncRunning:
-        folderMessage = tr("Sync is running.");
-        break;
-    case SyncResult::Success:
-        folderMessage = tr("Last Sync was successful.");
-        break;
-    case SyncResult::Error:
-        folderMessage = tr( "Syncing Error." );
-        break;
-    case SyncResult::SetupError:
-        folderMessage = tr( "Setup Error." );
-        break;
-    default:
-        folderMessage = tr( "Undefined Error State." );
-    }
-    folderMessage = QLatin1String("<b>") + folderMessage + QLatin1String("</b><br/>");
-
-    QMessageBox infoBox( QMessageBox::Information, tr( "Folder information" ), alias, QMessageBox::Ok );
-    QStringList li = folderResult.errorStrings();
-    foreach( const QString& l, li ) {
-        folderMessage += QString::fromLatin1("<p>%1</p>").arg( l );
-    }
-
-    infoBox.setText( folderMessage );
-
-    //    qDebug() << "informative text: " << infoBox.informativeText();
-
-    if ( !folderResult.syncChanges().isEmpty() ) {
-        QString details;
-        QHash < QString, QStringList > changes = folderResult.syncChanges();
-        QHash< QString, QStringList >::const_iterator change_it = changes.constBegin();
-        for(; change_it != changes.constEnd(); ++change_it ) {
-            QString changeType = tr( "Unknown" );
-            if ( change_it.key() == QLatin1String("changed") ) {
-            changeType = tr( "Changed files:\n" );
-            } else if ( change_it.key() == QLatin1String("added") ) {
-                changeType = tr( "Added files:\n" );
-            } else if ( change_it.key() == QLatin1String("deleted") ) {
-            changeType = tr( "New files in the server, or files deleted locally:\n");
-            }
-
-            QStringList files = change_it.value();
-            QString fileList;
-                foreach( const QString& file, files) {
-                    fileList += file + QLatin1Char('\n');
-            }
-            details += changeType + fileList;
-        }
-        infoBox.setDetailedText(details);
-        qDebug() << "detailed text: " << infoBox.detailedText();
-    }
-    infoBox.exec();
+    _fileItemDialog->setSyncResult( folderResult );
+    raiseDialog( _fileItemDialog );
 }
 
 void Application::slotEnableFolder(const QString& alias, const bool enable)
@@ -863,6 +845,11 @@ void Application::slotParseOptions(const QString &opts)
     setupLogBrowser();
 }
 
+void Application::slotShowTrayMessage(const QString &title, const QString &msg)
+{
+    _tray->showMessage(title, msg);
+}
+
 void Application::slotSyncStateChange( const QString& alias )
 {
     SyncResult result = _folderMan->syncResult( alias );
@@ -870,6 +857,10 @@ void Application::slotSyncStateChange( const QString& alias )
     // do not promote LocalSyncState to the status dialog.
     if( !result.localRunOnly() ) {
         _statusDialog->slotUpdateFolderState( _folderMan->folder(alias) );
+
+        if( _fileItemDialog && _fileItemDialog->isVisible() ) {
+            _fileItemDialog->setSyncResult( _folderMan->syncResult(alias) );
+        }
     }
     computeOverallSyncStatus();
 
@@ -897,6 +888,8 @@ void Application::parseOptions(const QStringList &options)
             }
         } else if (option == QLatin1String("--logflush")) {
             _logFlush = true;
+        } else if (option == QLatin1String("--monoicons")) {
+            _theme->setSystrayUseMonoIcons(true); 
         }
     }
 }
@@ -931,6 +924,10 @@ void Application::computeOverallSyncStatus()
                 folderMessage = tr( "Sync is running." );
                 overallResult.setStatus( SyncResult::SyncRunning );
                 break;
+            case SyncResult::Unavailable:
+                folderMessage = tr( "Server is currently not available." );
+                overallResult.setStatus( SyncResult::Unavailable );
+                break;
             case SyncResult::Success:
                 if( overallResult.status() == SyncResult::Undefined ) {
                     folderMessage = tr( "Last Sync was successful." );
@@ -972,7 +969,7 @@ void Application::computeOverallSyncStatus()
         else
             trayMessage = tr("No sync folders configured.");
 
-        QIcon statusIcon = _theme->syncStateIcon( overallResult.status()); // size 48 before
+        QIcon statusIcon = _theme->syncStateIcon( overallResult.status(), true); // size 48 before
 
         _tray->setIcon( statusIcon );
         _tray->setToolTip(trayMessage);
@@ -987,7 +984,8 @@ void Application::showHelp()
     std::cout << "Options:" << std::endl;
     std::cout << "  --logwindow          : open a window to show log output." << std::endl;
     std::cout << "  --logfile <filename> : write log output to file <filename>." << std::endl;
-    std::cout << "  --flushlog           : flush the log file after every write." << std::endl;
+    std::cout << "  --logflush           : flush the log file after every write." << std::endl;
+    std::cout << "  --monoicons          : Use black/white pictograms for systray." << std::endl;
     std::cout << std::endl;
     if (_theme->appName() == QLatin1String("ownCloud"))
         std::cout << "For more information, see http://www.owncloud.org" << std::endl;

src/mirall/application.h

@@ -26,6 +26,7 @@
 #include "mirall/folder.h"
 #include "mirall/logbrowser.h"
 #include "mirall/folderman.h"
+#include "mirall/fileitemdialog.h"
 
 class QAction;
 class QMenu;
@@ -64,8 +65,10 @@ protected slots:
     void slotConfigure();
     void slotConfigureProxy();
     void slotParseOptions( const QString& );
+    void slotShowTrayMessage(const QString&, const QString&);
 
     void slotSyncStateChange( const QString& );
+    void slotownCloudWizardDone(int);
 protected:
 
     void parseOptions(const QStringList& );
@@ -96,6 +99,7 @@ protected slots:
 
 private:
     void showHelp();
+    void raiseDialog( QWidget* );
 
     // configuration file -> folder
     QSystemTrayIcon *_tray;
@@ -118,6 +122,7 @@ private:
     // tray's menu
     QMenu *_contextMenu;
     StatusDialog *_statusDialog;
+    FileItemDialog *_fileItemDialog;
 
     FolderMan *_folderMan;
     Theme *_theme;
@@ -128,7 +133,7 @@ private:
     QString _logFile;
     bool _showLogWindow;
     bool _logFlush;
-    bool        _helpOnly;
+    bool _helpOnly;
 };
 
 } // namespace Mirall

src/mirall/credentialstore.cpp

@@ -13,17 +13,28 @@
 
 #include <QtGui>
 
+#include "config.h"
+
 #include "mirall/credentialstore.h"
 #include "mirall/mirallconfigfile.h"
 #include "mirall/theme.h"
 
+#define MAX_LOGIN_ATTEMPTS 3
+
 namespace Mirall {
 
 CredentialStore *CredentialStore::_instance=0;
 CredentialStore::CredState CredentialStore::_state = NotFetched;
-QString CredentialStore::_passwd = QString::null;
-QString CredentialStore::_user   = QString::null;
-int     CredentialStore::_tries  = 0;
+QString CredentialStore::_passwd   = QString::null;
+QString CredentialStore::_user     = QString::null;
+QString CredentialStore::_url      = QString::null;
+QString CredentialStore::_errorMsg = QString::null;
+int     CredentialStore::_tries    = 0;
+#ifdef WITH_QTKEYCHAIN
+CredentialStore::CredentialType CredentialStore::_type = KeyChain;
+#else
+CredentialStore::CredentialType CredentialStore::_type = Settings;
+#endif
 
 CredentialStore::CredentialStore(QObject *parent) :
     QObject(parent)
@@ -50,28 +61,63 @@ CredentialStore::CredState CredentialStore::state()
     return _state;
 }
 
+bool CredentialStore::canTryAgain()
+{
+    bool canDoIt = false;
+
+    if( _tries > MAX_LOGIN_ATTEMPTS ) {
+        qDebug() << "canTryAgain: Max attempts reached.";
+        return false;
+    }
+
+    if( _state == NotFetched ) {
+        return true;
+    }
+
+    switch( _type ) {
+    case CredentialStore::User:
+        canDoIt = true;
+        break;
+    case CredentialStore::Settings:
+        break;
+    case CredentialStore::KeyChain:
+        canDoIt = true;
+        break;
+    default:
+        break;
+    }
+    return canDoIt;
+}
+
 void CredentialStore::fetchCredentials()
 {
-    _state = Fetching;
     MirallConfigFile cfgFile;
-    MirallConfigFile::CredentialType t;
-
-    if( _tries++ == 3 ) {
+    if( ++_tries > MAX_LOGIN_ATTEMPTS ) {
         qDebug() << "Too many attempts to enter password!";
         _state = TooManyAttempts;
+        emit( fetchCredentialsFinished(false) );
         return;
     }
-    t = cfgFile.credentialType();
 
     bool ok = false;
     QString pwd;
-    _state = Fetching;
     _user = cfgFile.ownCloudUser();
+    _url  = cfgFile.ownCloudUrl();
 
-    switch( t ) {
-    case MirallConfigFile::User: {
+    QString key = keyChainKey(_url);
+
+    if( key.isNull() ) {
+        qDebug() << "Can not fetch credentials, url is zero!";
+        _state = Error;
+        emit( fetchCredentialsFinished(false) );
+        return;
+    }
+
+    switch( _type ) {
+    case CredentialStore::User: {
         /* Ask the user for the password */
         /* Fixme: Move user interaction out here. */
+        _state = Fetching;
         pwd = QInputDialog::getText(0, QApplication::translate("MirallConfigFile","Password Required"),
                                     QApplication::translate("MirallConfigFile","Please enter your %1 password:")
                                     .arg(Theme::instance()->appName()),
@@ -82,26 +128,33 @@ void CredentialStore::fetchCredentials()
         }
         break;
     }
-    case MirallConfigFile::Settings: {
+    case CredentialStore::Settings: {
         /* Read from config file. */
+        _state = Fetching;
         pwd = cfgFile.ownCloudPasswd();
         ok = true;
         break;
     }
-    case MirallConfigFile::KeyChain: {
-        /* Qt Keychain is not yet implemented. */
-#ifdef HAVE_QTKEYCHAIN
+    case CredentialStore::KeyChain: {
+        // If the credentials are here already, return.
+        if( _state == Ok ) {
+            emit(fetchCredentialsFinished(true));
+            return;
+        }
+        // otherwise fetch asynchronious.
+#ifdef WITH_QTKEYCHAIN
+        _state = AsyncFetching;
         if( !_user.isEmpty() ) {
-            ReadPasswordJoei b job( QLatin1String(Theme::instance()->appName()) );
-            job.setAutoDelete( false );
-            job.setKey( _user );
+            ReadPasswordJob *job = new ReadPasswordJob(Theme::instance()->appName());
+            job->setKey( key );
 
-            job.connect( &job, SIGNAL(finished(QKeychain::Job*)), this,
-                         SLOT(slotKeyChainFinished(QKeyChain::Job*)));
-            job.start();
+            connect( job, SIGNAL(finished(QKeychain::Job*)), this,
+                     SLOT(slotKeyChainReadFinished(QKeychain::Job*)));
+            job->start();
         }
 #else
         qDebug() << "QtKeyChain: Not yet implemented!";
+        _state = Error;
 #endif
         break;
     }
@@ -110,30 +163,123 @@ void CredentialStore::fetchCredentials()
     }
     }
 
-    if( ok ) {
-        _passwd = pwd;
-        _state = Ok;
-    }
-    if( !ok && _state == Fetching ) {
-        _state = Error;
-    }
-
-    emit( fetchCredentialsFinished(ok) );
-}
-
-#ifdef HAVE_QTKEYCHAIN
-void CredentialsStore::slotKeyChainFinished(QKeyChain::Job* job)
-{
-    if( job ) {
-        if( job->error() ) {
-            qDebug() << "Error mit keychain: " << job->errorString();
-        } else {
-            _passwd = job.textData();
+    if( _state == Fetching ) { // ...but not AsyncFetching
+        if( ok ) {
+            _passwd = pwd;
+            _state = Ok;
         }
+        if( !ok && _state == Fetching ) {
+            _state = Error;
+        }
+
+        emit( fetchCredentialsFinished(ok) );
+    } else {
+        // in case of AsyncFetching nothing happens here. The finished-Slot
+        // will emit the finish signal.
     }
 }
-#endif
 
+void CredentialStore::reset()
+{
+    _state = NotFetched;
+    _user   = QString::null;
+    _passwd = QString::null;
+    _tries = 0;
+}
+
+QString CredentialStore::keyChainKey( const QString& url ) const
+{
+    QString u(url);
+    if( u.isEmpty() ) {
+        qDebug() << "Empty url in keyChain, error!";
+        return QString::null;
+    }
+    if( _user.isEmpty() ) {
+        qDebug() << "Error: User is emty!";
+        return QString::null;
+    }
+
+    if( !u.endsWith(QChar('/')) ) {
+        u.append(QChar('/'));
+    }
+
+    QString key = _user+QLatin1Char(':')+u;
+    return key;
+}
+
+void CredentialStore::slotKeyChainReadFinished(QKeychain::Job* job)
+{
+#ifdef WITH_QTKEYCHAIN
+    ReadPasswordJob *pwdJob = static_cast<ReadPasswordJob*>(job);
+    if( pwdJob ) {
+        switch( pwdJob->error() ) {
+        case QKeychain::NoError:
+            _passwd = pwdJob->textData();
+#ifdef Q_OS_LINUX
+            // Currently there is a bug in the keychain on linux that if no
+            // entry is there, an empty password comes back, but no error.
+            if( _passwd.isEmpty() ) {
+                _state = EntryNotFound;
+                _errorMsg = tr("No password entry found in keychain. Please reconfigure.");
+            } else
+#endif
+            _state = Ok;
+            break;
+        case QKeychain::EntryNotFound:
+            _state = EntryNotFound;
+            break;
+        case QKeychain::CouldNotDeleteEntry:
+            _state = Error;
+            break;
+        case QKeychain::AccessDeniedByUser:
+           _state = AccessDeniedByUser;
+            break;
+        case QKeychain::AccessDenied:
+            _state = AccessDenied;
+            break;
+        case QKeychain::NoBackendAvailable:
+            _state = NoKeychainBackend;
+            break;
+        case QKeychain::NotImplemented:
+            _state = NoKeychainBackend;
+            break;
+        case QKeychain::OtherError:
+        default:
+            _state = Error;
+
+        }
+        /* In case there is no backend, tranparentely switch to Settings file. */
+        if( _state == NoKeychainBackend ) {
+            qDebug() << "No Storage Backend, falling back to Settings mode.";
+            _type = CredentialStore::Settings;
+            fetchCredentials();
+            return;
+        }
+
+        if( _state == EntryNotFound ) {
+            // try to migrate.
+        }
+
+        if( _state != Ok ) {
+            qDebug() << "Error with keychain: " << pwdJob->errorString();
+            if(_errorMsg.isEmpty()) _errorMsg = pwdJob->errorString();
+        } else {
+            _errorMsg = QString::null;
+        }
+    } else {
+        _state = Error;
+        qDebug() << "Error: KeyChain Read Password Job failed!";
+    }
+    emit(fetchCredentialsFinished(_state == Ok));
+#else
+    (void) job;
+#endif
+}
+
+QString CredentialStore::errorMessage()
+{
+    return _errorMsg;
+}
 
 QByteArray CredentialStore::basicAuthHeader() const
 {
@@ -144,11 +290,89 @@ QByteArray CredentialStore::basicAuthHeader() const
     return data;
 }
 
-void CredentialStore::setCredentials( const QString& user, const QString& pwd )
+void CredentialStore::setCredentials( const QString& url, const QString& user, const QString& pwd )
 {
     _passwd = pwd;
     _user = user;
+    _url  = url;
     _state = Ok;
 }
 
+void CredentialStore::saveCredentials( )
+{
+    MirallConfigFile cfgFile;
+    QString key = keyChainKey(_url);
+    if( key.isNull() ) {
+        qDebug() << "Error: Can not save credentials, URL is zero!";
+        return;
+    }
+#ifdef WITH_QTKEYCHAIN
+    WritePasswordJob *job = NULL;
+#endif
+
+    switch( _type ) {
+    case CredentialStore::User:
+        deleteKeyChainCredential( key );
+        break;
+    case CredentialStore::KeyChain:
+#ifdef WITH_QTKEYCHAIN
+        // Set password in KeyChain
+        job = new WritePasswordJob(Theme::instance()->appName());
+        job->setKey( key );
+        job->setTextData(_passwd);
+
+        connect( job, SIGNAL(finished(QKeychain::Job*)), this,
+                 SLOT(slotKeyChainWriteFinished(QKeychain::Job*)));
+        job->start();
+#endif
+        break;
+    case CredentialStore::Settings:
+        cfgFile.writePassword( _passwd );
+        reset();
+        break;
+    default:
+        // unsupported.
+        break;
+    }
+}
+
+void CredentialStore::slotKeyChainWriteFinished( QKeychain::Job *job )
+{
+#ifdef WITH_QTKEYCHAIN
+    WritePasswordJob *pwdJob = static_cast<WritePasswordJob*>(job);
+    if( pwdJob ) {
+        QKeychain::Error err = pwdJob->error();
+
+        if( err != QKeychain::NoError ) {
+            qDebug() << "Error with keychain: " << pwdJob->errorString();
+            if( err == NoBackendAvailable || err == NotImplemented ||
+                    pwdJob->errorString().contains(QLatin1String("Could not open wallet"))) {
+                _type = Settings;
+                saveCredentials();
+            }
+        } else {
+            qDebug() << "Successfully stored password for user " << _user;
+            // Try to remove password formerly stored in the config file.
+            MirallConfigFile cfgFile;
+            cfgFile.clearPasswordFromConfig();
+        }
+    } else {
+        qDebug() << "Error: KeyChain Write Password Job failed!";
+    }
+#else
+    (void) job;
+#endif
+}
+
+// Called if a user chooses to not store the password locally.
+void CredentialStore::deleteKeyChainCredential( const QString& key )
+{
+#ifdef WITH_QTKEYCHAIN
+    // Start the remove job, do not care so much about the result.
+    DeletePasswordJob *job = new DeletePasswordJob(Theme::instance()->appName());
+    job->setKey( key );
+    job->start();
+#endif
+}
+
 }

src/mirall/credentialstore.h

@@ -14,8 +14,22 @@
 #ifndef CREDENTIALSTORE_H
 #define CREDENTIALSTORE_H
 
+#include "config.h"
 #include <QObject>
 
+#ifdef WITH_QTKEYCHAIN
+#include "qtkeychain/keychain.h"
+
+using namespace QKeychain;
+#else
+// FIXME: If the slot definition below is ifdefed for some reason the slot is
+// not there even if WITH_QTKEYCHAIN is defined.
+namespace QKeychain {
+   typedef void Job;
+}
+#endif
+
+
 namespace Mirall {
 
 /*
@@ -42,7 +56,23 @@ class CredentialStore : public QObject
 {
     Q_OBJECT
 public:
-    enum CredState { NotFetched = 0, Ok, UserCanceled, Fetching, Error, TooManyAttempts };
+    enum CredState { NotFetched = 0,
+                     Ok,
+                     UserCanceled,
+                     Fetching,
+                     AsyncFetching,
+                     EntryNotFound,
+                     AccessDeniedByUser,
+                     AccessDenied,
+                     NoKeychainBackend,
+                     Error,
+                     TooManyAttempts };
+
+    enum CredentialType {
+        User = 0,
+        Settings,
+        KeyChain
+    };
 
     QString password( const QString& connection = QString::null ) const;
     QString user( const QString& connection = QString::null ) const;
@@ -77,11 +107,23 @@ public:
      * @brief setCredentials - sets the user credentials.
      *
      * This function is called from the setup wizard to set the credentials
-     * int this store. The function also sets the state to ok.
+     * int this store. Note that it does not store the password.
+     * The function also sets the state to ok.
      * @param user - the user name
      * @param password - the password.
      */
-    void setCredentials( const QString&, const QString& );
+    void setCredentials( const QString&, const QString&, const QString& );
+
+    void saveCredentials( );
+
+    QString errorMessage();
+
+    /**
+     * @brief canTryAgain - check if another try to get credentials makes sense.
+    */
+    bool canTryAgain();
+
+    void reset();
 signals:
     /**
      * @brief fetchCredentialsFinished
@@ -93,19 +135,23 @@ signals:
      */
     void fetchCredentialsFinished(bool);
 
-private slots:
-#ifdef HAVE_QTKEYCHAIN
-void slotKeyChainFinished(QKeyChain::Job* job);
-#endif
+protected slots:
+    void slotKeyChainReadFinished( QKeychain::Job* );
+    void slotKeyChainWriteFinished( QKeychain::Job* );
 
 private:
     explicit CredentialStore(QObject *parent = 0);
+    void deleteKeyChainCredential( const QString& );
+    QString keyChainKey( const QString& ) const;
 
     static CredentialStore *_instance;
     static CredState _state;
     static QString _passwd;
     static QString _user;
+    static QString _url;
+    static QString _errorMsg;
     static int     _tries;
+    static CredentialType _type;
 };
 }
 

src/mirall/csyncfolder.cpp

@@ -87,7 +87,7 @@ void CSyncFolder::slotCSyncFinished()
     SyncResult res(SyncResult::Success);
     if( _csyncError ) {
         res.setStatus( SyncResult::Error );
-        res.setErrorString( _errors.join(QLatin1String("\\n")));
+        res.setErrorString( _errors.join(QLatin1String("\n")));
     }
     emit syncFinished( res );
 }

src/mirall/csyncthread.cpp

@@ -16,8 +16,18 @@
 #include "mirall/csyncthread.h"
 #include "mirall/mirallconfigfile.h"
 #include "mirall/theme.h"
+#include "mirall/logger.h"
+#include "mirall/utility.h"
+#include "mirall/owncloudinfo.h"
+
+#ifdef Q_OS_WIN
+#include <windows.h>
+#else
+#include <unistd.h>
+#endif
 
 #include <QDebug>
+#include <QSslSocket>
 #include <QDir>
 #include <QMutexLocker>
 #include <QThread>
@@ -25,6 +35,8 @@
 #include <QTextStream>
 #include <QTime>
 #include <QApplication>
+#include <QUrl>
+#include <QSslCertificate>
 
 namespace Mirall {
 
@@ -37,122 +49,18 @@ QString CSyncThread::_csyncConfigDir;  // to be able to remove the lock file.
 
 QMutex CSyncThread::_mutex;
 
-struct proxyInfo_s {
-    char *proxyType;
-    char *proxyHost;
-    char *proxyPort;
-    char *proxyUser;
-    char *proxyPwd;
-};
-typedef proxyInfo_s ProxyInfo;
-
-walkStats_s::walkStats_s() {
-    errorType = 0;
-
-    eval = 0;
-    removed = 0;
-    renamed = 0;
-    newFiles = 0;
-    conflicts = 0;
-    ignores = 0;
-    sync = 0;
-    error = 0;
-
-    dirPermErrors = 0;
-
-    seenFiles = 0;
-
-}
-
-
-int CSyncThread::recordStats( TREE_WALK_FILE* file )
+void csyncLogCatcher(CSYNC *ctx,
+                     int verbosity,
+                     const char *function,
+                     const char *buffer,
+                     void *userdata)
 {
-    if( ! file ) return -1;
-    _mutex.lock();
-
-    _walkStats.seenFiles++;
-
-    switch(file->instruction) {
-    case CSYNC_INSTRUCTION_NONE:
-
-        break;
-    case CSYNC_INSTRUCTION_EVAL:
-        _walkStats.eval++;
-        break;
-    case CSYNC_INSTRUCTION_REMOVE:
-        _walkStats.removed++;
-        break;
-    case CSYNC_INSTRUCTION_RENAME:
-        _walkStats.renamed++;
-        break;
-    case CSYNC_INSTRUCTION_NEW:
-        _walkStats.newFiles++;
-        break;
-    case CSYNC_INSTRUCTION_CONFLICT:
-        _walkStats.conflicts++;
-        break;
-    case CSYNC_INSTRUCTION_IGNORE:
-        _walkStats.ignores++;
-        break;
-    case CSYNC_INSTRUCTION_SYNC:
-        _walkStats.sync++;
-        break;
-    case CSYNC_INSTRUCTION_STAT_ERROR:
-    case CSYNC_INSTRUCTION_ERROR:
-        /* instructions for the propagator */
-    case CSYNC_INSTRUCTION_DELETED:
-    case CSYNC_INSTRUCTION_UPDATED:
-        _walkStats.error++;
-        _walkStats.errorType = WALK_ERROR_INSTRUCTIONS;
-        break;
-    default:
-        _walkStats.error++;
-        _walkStats.errorType = WALK_ERROR_WALK;
-        break;
-    }
-
-    int re = 0;
-    // qDebug() << _walkStats.seenFiles << ". Path: " << file->path << ": uid= " << file->uid << " - type: " << file->type;
-    if( !( _walkStats.errorType == WALK_ERROR_NONE || _walkStats.errorType == WALK_ERROR_DIR_PERMS )) {
-        re = -1;
-    }
-    _mutex.unlock();
-
-    return re;
+  Logger::instance()->csyncLog( QString::fromUtf8(buffer) );
 }
 
-int CSyncThread::treewalk( TREE_WALK_FILE* file, void *data )
-{
-    int re = static_cast<CSyncThread*>(data)->recordStats( file );
-    if( re > -1 )
-        return static_cast<CSyncThread*>(data)->treewalkFile( file );
-    return -1;
-}
-
-int CSyncThread::treewalkFile( TREE_WALK_FILE *file )
-{
-    if( ! file ) return -1;
-    SyncFileItem item;
-    item.file = QString::fromUtf8( file->path );
-    item.instruction = file->instruction;
-
-    QFileInfo fi( _source, item.file );
-    if( !(fi.isWritable() && fi.isExecutable()) ) {
-        _walkStats.dirPermErrors++;
-    }
-
-    _mutex.lock();
-    _syncedItems.append(item);
-    _mutex.unlock();
-
-    return 0;
-}
-
-CSyncThread::CSyncThread(const QString &source, const QString &target, bool localCheckOnly)
-
+CSyncThread::CSyncThread(const QString &source, const QString &target)
     : _source(source)
     , _target(target)
-    , _localCheckOnly( localCheckOnly )
 
 {
     _mutex.lock();
@@ -165,43 +73,226 @@ CSyncThread::~CSyncThread()
 
 }
 
-static char* proxyTypeToCStr(QNetworkProxy::ProxyType type)
+QString CSyncThread::csyncErrorToString( CSYNC_ERROR_CODE err, const char *errString )
+{
+    QString errStr;
+
+    switch( err ) {
+    case CSYNC_ERR_NONE:
+        errStr = tr("Success.");
+        break;
+    case CSYNC_ERR_LOG:
+        errStr = tr("CSync Logging setup failed.");
+        break;
+    case CSYNC_ERR_LOCK:
+        errStr = tr("CSync failed to create a lock file.");
+        break;
+    case CSYNC_ERR_STATEDB_LOAD:
+        errStr = tr("CSync failed to load the state db.");
+        break;
+    case CSYNC_ERR_MODULE:
+        errStr = tr("<p>The %1 plugin for csync could not be loaded.<br/>Please verify the installation!</p>").arg(Theme::instance()->appName());
+        break;
+    case CSYNC_ERR_TIMESKEW:
+        errStr = tr("The system time on this client is different than the system time on the server. "
+                    "Please use a time synchronization service (NTP) on the server and client machines "
+                    "so that the times remain the same.");
+        break;
+    case CSYNC_ERR_FILESYSTEM:
+        errStr = tr("CSync could not detect the filesystem type.");
+        break;
+    case CSYNC_ERR_TREE:
+        errStr = tr("CSync got an error while processing internal trees.");
+        break;
+    case CSYNC_ERR_MEM:
+        errStr = tr("CSync failed to reserve memory.");
+        break;
+    case CSYNC_ERR_PARAM:
+        errStr = tr("CSync fatal parameter error.");
+        break;
+    case CSYNC_ERR_UPDATE:
+        errStr = tr("CSync processing step update failed.");
+        break;
+    case CSYNC_ERR_RECONCILE:
+        errStr = tr("CSync processing step reconcile failed.");
+        break;
+    case CSYNC_ERR_PROPAGATE:
+        errStr = tr("CSync processing step propagate failed.");
+        break;
+    case CSYNC_ERR_ACCESS_FAILED:
+        errStr = tr("<p>The target directory %1 does not exist.</p><p>Please check the sync setup.</p>").arg(_target);
+        // this is critical. The database has to be removed.
+        emit wipeDb();
+        break;
+    case CSYNC_ERR_REMOTE_CREATE:
+    case CSYNC_ERR_REMOTE_STAT:
+        errStr = tr("A remote file can not be written. Please check the remote access.");
+        break;
+    case CSYNC_ERR_LOCAL_CREATE:
+    case CSYNC_ERR_LOCAL_STAT:
+        errStr = tr("The local filesystem can not be written. Please check permissions.");
+        break;
+    case CSYNC_ERR_PROXY:
+        errStr = tr("CSync failed to connect through a proxy.");
+        break;
+    case CSYNC_ERR_LOOKUP:
+        errStr = tr("CSync failed to lookup proxy or server.");
+        break;
+    case CSYNC_ERR_AUTH_SERVER:
+        errStr = tr("CSync failed to authenticate at the %1 server.").arg(Theme::instance()->appName());
+        break;
+    case CSYNC_ERR_AUTH_PROXY:
+        errStr = tr("CSync failed to authenticate at the proxy.");
+        break;
+    case CSYNC_ERR_CONNECT:
+        errStr = tr("CSync failed to connect to the network.");
+        break;
+    case CSYNC_ERR_TIMEOUT:
+        errStr = tr("A network connection timeout happend.");
+        break;
+    case CSYNC_ERR_HTTP:
+        errStr = tr("A HTTP transmission error happened.");
+        break;
+    case CSYNC_ERR_PERM:
+        errStr = tr("CSync failed due to not handled permission deniend.");
+        break;
+    case CSYNC_ERR_NOT_FOUND:
+        errStr = tr("CSync failed to find a specific file.");
+        break;
+    case CSYNC_ERR_EXISTS:
+        errStr = tr("CSync tried to create a directory that already exists.");
+        break;
+    case CSYNC_ERR_NOSPC:
+        errStr = tr("CSync: No space on %1 server available.").arg(Theme::instance()->appName());
+        break;
+    case CSYNC_ERR_UNSPEC:
+        errStr = tr("CSync unspecified error.");
+
+    default:
+        errStr = tr("An internal error number %1 happend.").arg( (int) err );
+    }
+
+    if( errString ) {
+        errStr += tr("<br/>Backend Message: ")+QString::fromUtf8(errString);
+    }
+    return errStr;
+
+}
+
+const char* CSyncThread::proxyTypeToCStr(QNetworkProxy::ProxyType type)
 {
     switch (type) {
     case QNetworkProxy::NoProxy:
-        return qstrdup("NoProxy");
+        return "NoProxy";
     case QNetworkProxy::DefaultProxy:
-        return qstrdup("DefaultProxy");
+        return "DefaultProxy";
     case QNetworkProxy::Socks5Proxy:
-        return qstrdup("Socks5Proxy");
-    case  QNetworkProxy::HttpProxy:
-        return qstrdup("HttpProxy");
-    case  QNetworkProxy::HttpCachingProxy:
-        return qstrdup("HttpCachingProxy");
-    case  QNetworkProxy::FtpCachingProxy:
-        return qstrdup("FtpCachingProxy");
+        return "Socks5Proxy";
+    case QNetworkProxy::HttpProxy:
+        return "HttpProxy";
+    case QNetworkProxy::HttpCachingProxy:
+        return "HttpCachingProxy";
+    case QNetworkProxy::FtpCachingProxy:
+        return "FtpCachingProxy";
     default:
-        return qstrdup("NoProxy");
+        return "NoProxy";
     }
 }
 
+int CSyncThread::treewalkLocal( TREE_WALK_FILE* file, void *data )
+{
+    return static_cast<CSyncThread*>(data)->treewalkFile( file, false );
+}
+
+int CSyncThread::treewalkRemote( TREE_WALK_FILE* file, void *data )
+{
+    return static_cast<CSyncThread*>(data)->treewalkFile( file, true );
+}
+
+int CSyncThread::walkFinalize(TREE_WALK_FILE* file, void *data )
+{
+    return static_cast<CSyncThread*>(data)->treewalkError( file);
+}
+
+int CSyncThread::treewalkFile( TREE_WALK_FILE *file, bool remote )
+{
+    if( ! file ) return -1;
+    SyncFileItem item;
+    item._file = QString::fromUtf8( file->path );
+    item._instruction = file->instruction;
+    item._dir = SyncFileItem::None;
+
+    SyncFileItem::Direction dir;
+
+    int re = 0;
+
+    switch(file->instruction) {
+    case CSYNC_INSTRUCTION_NONE:
+        // No need to do anything.
+        return re;
+        break;
+    case CSYNC_INSTRUCTION_RENAME:
+        dir = !remote ? SyncFileItem::Down : SyncFileItem::Up;
+        item._renameTarget = QString::fromUtf8( file->rename_path );
+        break;
+    case CSYNC_INSTRUCTION_REMOVE:
+        dir = !remote ? SyncFileItem::Down : SyncFileItem::Up;
+        break;
+    case CSYNC_INSTRUCTION_CONFLICT:
+    case CSYNC_INSTRUCTION_IGNORE:
+    case CSYNC_INSTRUCTION_ERROR:
+        dir = SyncFileItem::None;
+        break;
+    case CSYNC_INSTRUCTION_EVAL:
+    case CSYNC_INSTRUCTION_NEW:
+    case CSYNC_INSTRUCTION_SYNC:
+    case CSYNC_INSTRUCTION_STAT_ERROR:
+    case CSYNC_INSTRUCTION_DELETED:
+    case CSYNC_INSTRUCTION_UPDATED:
+    default:
+        dir = remote ? SyncFileItem::Down : SyncFileItem::Up;
+        break;
+    }
+
+    item._dir = dir;
+    _mutex.lock();
+    _syncedItems.append(item);
+    _mutex.unlock();
+
+    return re;
+}
+
+int CSyncThread::treewalkError(TREE_WALK_FILE* file)
+{
+    SyncFileItem item;
+    item._file= QString::fromUtf8(file->path);
+    int indx = _syncedItems.indexOf(item);
+
+    if ( indx == -1 )
+        return 0;
+
+    if( item._instruction == CSYNC_INSTRUCTION_STAT_ERROR ||
+            item._instruction == CSYNC_INSTRUCTION_ERROR ) {
+        _mutex.lock();
+        _syncedItems[indx]._instruction = item._instruction;
+        _mutex.unlock();
+    }
+
+    return 0;
+}
+
+
 void CSyncThread::startSync()
 {
     qDebug() << "starting to sync " << qApp->thread() << QThread::currentThread();
     CSYNC *csync;
-    QTime walkTime;
-
-    ProxyInfo proxyInfo;
+    bool doTreeWalk = true;
+    int proxyPort = _proxy.port();
 
     emit(started());
 
     _mutex.lock();
-
-    proxyInfo.proxyType = proxyTypeToCStr( _proxy.type() );
-    proxyInfo.proxyHost = qstrdup( _proxy.hostName().toAscii().constData() );
-    proxyInfo.proxyPort = qstrdup( QByteArray::number( _proxy.port() ).constData() );
-    proxyInfo.proxyUser = qstrdup( _proxy.user().toAscii().constData() );
-    proxyInfo.proxyPwd  = qstrdup( _proxy.password().toAscii().constData() );
+    _syncedItems.clear();
 
     if( csync_create(&csync,
                      _source.toUtf8().data(),
@@ -211,17 +302,14 @@ void CSyncThread::startSync()
     _csyncConfigDir = QString::fromUtf8( csync_get_config_dir( csync ));
     _mutex.unlock();
 
-    qDebug() << "## CSync Thread local only: " << _localCheckOnly;
-    csync_set_auth_callback( csync, getauth );
     csync_enable_conflictcopys(csync);
 
-
     MirallConfigFile cfg;
     QString excludeList = cfg.excludeFile();
 
     if( !excludeList.isEmpty() ) {
-        qDebug() << "==== added CSync exclude List: " << excludeList.toAscii();
-        csync_add_exclude_list( csync, excludeList.toAscii() );
+        qDebug() << "==== added CSync exclude List: " << excludeList.toUtf8();
+        csync_add_exclude_list( csync, excludeList.toUtf8() );
     }
 
     csync_set_config_dir( csync, cfg.configPath().toUtf8() );
@@ -229,157 +317,92 @@ void CSyncThread::startSync()
     QTime t;
     t.start();
 
-    _mutex.lock();
-    if( _localCheckOnly ) {
-        csync_set_local_only( csync, true );
-    }
-    csync_set_userdata(csync, (void*) &proxyInfo);
-    _mutex.unlock();
+    csync_set_userdata(csync, this);
+
+    csync_set_log_callback( csync, csyncLogCatcher );
+    csync_set_auth_callback( csync, getauth );
+    csync_set_progress_callback( csync, progress );
 
     if( csync_init(csync) < 0 ) {
         CSYNC_ERROR_CODE err = csync_get_error( csync );
-        QString errStr;
-
-        switch( err ) {
-        case CSYNC_ERR_LOCK:
-            errStr = tr("CSync failed to create a lock file.");
-            break;
-        case CSYNC_ERR_STATEDB_LOAD:
-            errStr = tr("CSync failed to load the state db.");
-            break;
-        case CSYNC_ERR_TIMESKEW:
-            errStr = tr("The system time on this client is different than the system time on the server. "
-                        "Please use a time synchronization service (NTP) on the server and client machines "
-                        "so that the times remain the same.");
-            break;
-        case CSYNC_ERR_FILESYSTEM:
-            errStr = tr("CSync could not detect the filesystem type.");
-            break;
-        case CSYNC_ERR_TREE:
-            errStr = tr("CSync got an error while processing internal trees.");
-            break;
-        case CSYNC_ERR_ACCESS_FAILED:
-            errStr = tr("<p>The target directory %1 does not exist.</p><p>Please check the sync setup.</p>").arg(_target);
-            // this is critical. The database has to be removed.
-            emitStateDb(csync); // to make the name of the csync db known.
-            emit wipeDb();
-            break;
-        case CSYNC_ERR_MODULE:
-            errStr = tr("<p>The %1 plugin for csync could not be loaded.<br/>Please verify the installation!</p>").arg(Theme::instance()->appName());
-            break;
-        case CSYNC_ERR_LOCAL_CREATE:
-        case CSYNC_ERR_LOCAL_STAT:
-            errStr = tr("The local filesystem can not be written. Please check permissions.");
-            break;
-        case CSYNC_ERR_REMOTE_CREATE:
-        case CSYNC_ERR_REMOTE_STAT:
-            errStr = tr("A remote file can not be written. Please check the remote access.");
-            break;
-        default:
-            errStr = tr("An internal error number %1 happend.").arg( (int) err );
-        }
-        qDebug() << " #### ERROR String emitted: " << errStr;
+        const char *errMsg = csync_get_error_string( csync );
+        QString errStr = csyncErrorToString(err, errMsg );
+        qDebug() << " #### ERROR csync_init: " << errStr;
         emit csyncError(errStr);
         goto cleanup;
     }
 
-    emitStateDb(csync);
+    // set module properties, mainly the proxy information.
+    // do not use QLatin1String here because that has to be real const char* for C.
+    csync_set_log_verbosity(csync, 11);
+    csync_set_module_property(csync, "csync_context", csync);
+    csync_set_module_property(csync, "proxy_type", (char*) proxyTypeToCStr(_proxy.type()) );
+    csync_set_module_property(csync, "proxy_host", _proxy.hostName().toUtf8().data() );
+    csync_set_module_property(csync, "proxy_port", &proxyPort );
+    csync_set_module_property(csync, "proxy_user", _proxy.user().toUtf8().data()     );
+    csync_set_module_property(csync, "proxy_pwd" , _proxy.password().toUtf8().data() );
 
     qDebug() << "#### Update start #################################################### >>";
     if( csync_update(csync) < 0 ) {
         CSYNC_ERROR_CODE err = csync_get_error( csync );
-        QString errStr;
-
-        switch( err ) {
-        case CSYNC_ERR_PROXY:
-            errStr = tr("CSync failed to reach the host. Either host or proxy settings are not valid.");
-            break;
-        default:
-            errStr = tr("CSync Update failed.");
-            break;
-        }
-        emit csyncError( errStr );
+        const char *errMsg = csync_get_error_string( csync );
+        QString errStr = csyncErrorToString(err, errMsg);
+        qDebug() << " #### ERROR csync_update: " << errStr;
+        if (err == CSYNC_ERR_SERVICE_UNAVAILABLE)
+            emit csyncUnavailable();
+        else
+            emit csyncError(errStr);
         goto cleanup;
     }
     qDebug() << "<<#### Update end ###########################################################";
 
-    csync_set_userdata(csync, this);
 
-    walkTime.start();
-    if( csync_walk_local_tree(csync, &treewalk, 0) < 0 ) {
-        qDebug() << "Error in treewalk.";
-        if( _walkStats.errorType == WALK_ERROR_WALK ) {
-            emit csyncError(tr("CSync encountered an error while examining the file system.\n"
-                               "Syncing is not possible."));
-        } else if( _walkStats.errorType == WALK_ERROR_INSTRUCTIONS ) {
-            emit csyncError(tr("CSync update generated a strange instruction.\n"
-                               "Please write a bug report."));
-        }
-        emit csyncError(tr("Local filesystem problems. Better disable Syncing and check."));
+
+    if( csync_reconcile(csync) < 0 ) {
+        CSYNC_ERROR_CODE err = csync_get_error( csync );
+        const char *errMsg = csync_get_error_string( csync );
+        QString errStr = csyncErrorToString(err, errMsg);
+        qDebug() << " #### ERROR csync_reconcile: " << errStr;
+        emit csyncError(errStr);
         goto cleanup;
-    } else {
-        // only warn, do not stop the sync process.
-        if( _walkStats.errorType == WALK_ERROR_DIR_PERMS ) {
-            emit csyncError(tr("The local filesystem has %1 write protected directories."
-                               "That can hinder successful syncing.<p/>"
-                               "Please make sure that all local directories are writeable.").arg(_walkStats.dirPermErrors));
-        }
     }
 
-    // emit the treewalk results.
-    emit treeWalkResult(_syncedItems, _walkStats);
-
-    _mutex.lock();
-    if( _localCheckOnly ) {
-        _mutex.unlock();
-        qDebug() << " ..... Local only walk finished: " << walkTime.elapsed();
-        // we have to go out here as its local check only.
-        goto cleanup;
-    } else {
-        _mutex.unlock();
-        // check if we can write all over.
-
-        if( csync_reconcile(csync) < 0 ) {
-            emit csyncError(tr("CSync reconcile failed."));
-            goto cleanup;
-        }
-        if( csync_propagate(csync) < 0 ) {
-            emit csyncError(tr("File exchange with ownCloud failed. Sync was stopped."));
-            goto cleanup;
-        }
+    if( csync_walk_local_tree(csync, &treewalkLocal, 0) < 0 ) {
+         qDebug() << "Error in local treewalk.";
+         doTreeWalk = false;
     }
+    if( doTreeWalk && csync_walk_remote_tree(csync, &treewalkRemote, 0) < 0 ) {
+         qDebug() << "Error in remote treewalk.";
+         doTreeWalk = false;
+    }
+
+    if( csync_propagate(csync) < 0 ) {
+        CSYNC_ERROR_CODE err = csync_get_error( csync );
+        const char *errMsg = csync_get_error_string( csync );
+        QString errStr = csyncErrorToString(err, errMsg);
+        qDebug() << " #### ERROR csync_propagate: " << errStr;
+        if (err == CSYNC_ERR_SERVICE_UNAVAILABLE)
+            emit csyncUnavailable();
+        else
+            emit csyncError(errStr);
+        goto cleanup;
+    }
+
+    if( csync_walk_local_tree(csync, &walkFinalize, 0) < 0 ||
+            csync_walk_remote_tree( csync, &walkFinalize, 0 ) < 0 ) {
+        qDebug() << "Error in finalize treewalk.";
+    } else {
+        // emit the treewalk results.
+        emit treeWalkResult(_syncedItems);
+    }
+
 cleanup:
     csync_destroy(csync);
 
-    if( proxyInfo.proxyType ) delete[] proxyInfo.proxyType;
-    if( proxyInfo.proxyHost ) delete[] proxyInfo.proxyHost;
-    if( proxyInfo.proxyPort ) delete[] proxyInfo.proxyPort;
-    if( proxyInfo.proxyUser ) delete[] proxyInfo.proxyUser;
-    if( proxyInfo.proxyPwd  ) delete[] proxyInfo.proxyPwd ;
-
-    /*
-     * Attention: do not delete the wStat memory here. it is deleted in the
-     * slot catching the signel treeWalkResult because this thread can faster
-     * die than the slot has read out the data.
-     */
     qDebug() << "CSync run took " << t.elapsed() << " Milliseconds";
     emit(finished());
 }
 
-void CSyncThread::emitStateDb( CSYNC *csync )
-{
-    // After csync_init the statedb file name can be emitted
-    const char *statedb = csync_get_statedb_file( csync );
-    if( statedb ) {
-        QString stateDbFile = QString::fromUtf8(statedb);
-        free((void*)statedb);
-
-        emit csyncStateDbFile( stateDbFile );
-    } else {
-        qDebug() << "WRN: Unable to get csync statedb file name";
-    }
-}
-
 void CSyncThread::setConnectionDetails( const QString &user, const QString &passwd, const QNetworkProxy &proxy )
 {
     _mutex.lock();
@@ -404,27 +427,69 @@ int CSyncThread::getauth(const char *prompt,
 {
     int re = 0;
 
-    QString qPrompt = QString::fromLocal8Bit( prompt ).trimmed();
-    _mutex.lock();
+    QString qPrompt = QString::fromLatin1( prompt ).trimmed();
 
     if( qPrompt == QLatin1String("Enter your username:") ) {
         // qDebug() << "OOO Username requested!";
+        QMutexLocker locker( &_mutex );
         qstrncpy( buf, _user.toUtf8().constData(), len );
     } else if( qPrompt == QLatin1String("Enter your password:") ) {
+        QMutexLocker locker( &_mutex );
         // qDebug() << "OOO Password requested!";
         qstrncpy( buf, _passwd.toUtf8().constData(), len );
     } else {
         if( qPrompt.startsWith( QLatin1String("There are problems with the SSL certificate:"))) {
             // SSL is requested. If the program came here, the SSL check was done by mirall
-            // the answer is simply yes here.
-            qstrcpy( buf, "yes" );
+            // It needs to be checked if the  chain is still equal to the one which
+            // was verified by the user.
+            QRegExp regexp("fingerprint: ([\\w\\d:]+)");
+            bool certOk = false;
+
+            int pos = 0;
+
+
+            // This is the set of certificates which QNAM accepted, so we should accept
+            // them as well
+            QList<QSslCertificate> certs = ownCloudInfo::instance()->certificateChain();
+
+            while (!certOk && (pos = regexp.indexIn(qPrompt, 1+pos)) != -1) {
+                QString neon_fingerprint = regexp.cap(1);
+
+                foreach( const QSslCertificate& c, certs ) {
+                    QString verified_shasum = Utility::formatFingerprint(c.digest(QCryptographicHash::Sha1).toHex());
+                    qDebug() << "SSL Fingerprint from neon: " << neon_fingerprint << " compared to verified: " << verified_shasum;
+                    if( verified_shasum == neon_fingerprint ) {
+                        certOk = true;
+                        break;
+                    }
+                }
+            }
+            // certOk = false;     DEBUG setting, keep disabled!
+            if( !certOk ) { // Problem!
+                qstrcpy( buf, "no" );
+                re = -1;
+            } else {
+                qstrcpy( buf, "yes" ); // Certificate is fine!
+            }
         } else {
             qDebug() << "Unknown prompt: <" << prompt << ">";
             re = -1;
         }
     }
-    _mutex.unlock();
     return re;
 }
 
+
+void CSyncThread::progress(const char *remote_url, enum csync_notify_type_e kind,
+                                        long long o1, long long o2, void *userdata)
+{
+    (void) o1; (void) o2;
+    if (kind == CSYNC_NOTIFY_FINISHED_DOWNLOAD) {
+        QString path = QUrl::fromEncoded(remote_url).toString();
+        CSyncThread *thread = static_cast<CSyncThread*>(userdata);
+        thread->fileReceived(path);
+    }
+}
+
+
 }

src/mirall/csyncthread.h

@@ -25,61 +25,34 @@
 
 #include <csync.h>
 
+#include "mirall/syncfileitem.h"
+
 class QProcess;
 
 namespace Mirall {
 
-enum walkErrorTypes {
-    WALK_ERROR_NONE = 0,
-    WALK_ERROR_WALK,
-    WALK_ERROR_INSTRUCTIONS,
-    WALK_ERROR_DIR_PERMS
-};
-
-struct walkStats_s {
-    walkStats_s();
-
-    int errorType;
-
-    ulong eval;
-    ulong removed;
-    ulong renamed;
-    ulong newFiles;
-    ulong conflicts;
-    ulong ignores;
-    ulong sync;
-    ulong error;
-
-    ulong dirPermErrors;
-
-    ulong seenFiles;
-};
-
-typedef walkStats_s WalkStats;
-
-struct syncFileItem_s {
-    QString file;
-    csync_instructions_e instruction;
-};
-typedef syncFileItem_s SyncFileItem;
-
-typedef QVector<SyncFileItem> SyncFileItemVector;
-
 class CSyncThread : public QObject
 {
     Q_OBJECT
 public:
-    CSyncThread(const QString &source, const QString &target, bool = false);
+    CSyncThread(const QString &source, const QString &target);
     ~CSyncThread();
 
     static void setConnectionDetails( const QString&, const QString&, const QNetworkProxy& );
     static QString csyncConfigDir();
 
+    const char* proxyTypeToCStr(QNetworkProxy::ProxyType);
+    QString csyncErrorToString( CSYNC_ERROR_CODE, const char * );
+
     Q_INVOKABLE void startSync();
 
 signals:
-    void treeWalkResult(const SyncFileItemVector&, const WalkStats&);
+    void fileReceived( const QString& );
+    void fileRemoved( const QString& );
     void csyncError( const QString& );
+    void csyncWarning( const QString& );
+    void csyncUnavailable();
+    void treeWalkResult(const SyncFileItemVector&);
 
     void csyncStateDbFile( const QString& );
     void wipeDb();
@@ -88,10 +61,17 @@ signals:
     void started();
 
 private:
-    static int treewalk( TREE_WALK_FILE* file, void *data );
-    int recordStats( TREE_WALK_FILE* file);
-    void emitStateDb( CSYNC *csync );
-    int treewalkFile( TREE_WALK_FILE* );
+    static void progress(const char *remote_url,
+                    enum csync_notify_type_e kind,
+                    long long o1, long long o2,
+                    void *userdata);
+
+    static int treewalkLocal( TREE_WALK_FILE*, void *);
+    static int treewalkRemote( TREE_WALK_FILE*, void *);
+    int treewalkFile( TREE_WALK_FILE*, bool );
+    int treewalkError( TREE_WALK_FILE* );
+
+    static int walkFinalize(TREE_WALK_FILE*, void* );
 
     static int getauth(const char *prompt,
                 char *buf,
@@ -108,12 +88,10 @@ private:
 
     static QString _csyncConfigDir;
 
+    SyncFileItemVector _syncedItems;
+
     QString _source;
     QString _target;
-    bool    _localCheckOnly;
-
-    QVector <SyncFileItem> _syncedItems;
-    WalkStats _walkStats;
 };
 }
 

src/mirall/fileitemdialog.cpp

@@ -0,0 +1,295 @@
+/*
+ * Copyright (C) by Klaas Freitag <freitag@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#include <QtGui>
+
+#include "mirall/fileitemdialog.h"
+#include "mirall/theme.h"
+#include "mirall/syncresult.h"
+
+#define TYPE_SUCCESS  1
+#define TYPE_CONFLICT 2
+#define TYPE_NEW      3
+#define TYPE_DELETED  4
+#define TYPE_ERROR    5
+#define TYPE_RENAME   6
+#define TYPE_IGNORE   7
+
+#define FILE_TYPE    100
+
+namespace Mirall {
+
+FileItemDialog::FileItemDialog(Theme *theme, QWidget *parent) :
+    QDialog(parent),
+    _theme(theme)
+{
+    setupUi(this);
+    connect(_dialogButtonBox->button(QDialogButtonBox::Close), SIGNAL(clicked()),
+            this, SLOT(accept()));
+
+    QStringList header;
+    header << tr("Files");
+    QString firstColString = tr("File Count");
+    header << firstColString;
+    _treeWidget->setHeaderLabels( header );
+
+    _treeWidget->setColumnWidth(0, 480);
+    _timer.setInterval(1000);
+    connect(&_timer, SIGNAL(timeout()), this, SLOT(slotSetFolderMessage()));
+
+    QPushButton *copyBtn = _dialogButtonBox->addButton(tr("Copy"), QDialogButtonBox::ActionRole);
+    connect(copyBtn, SIGNAL(clicked()), SLOT(copyToClipboard()));
+
+    setWindowTitle(tr("Sync Protocol"));
+
+}
+
+void FileItemDialog::setSyncResult( const SyncResult& result )
+{
+    QString folderMessage;
+
+    SyncResult::Status syncStatus = result.status();
+    switch( syncStatus ) {
+    case SyncResult::Undefined:
+        folderMessage = tr( "Undefined Folder State" );
+        break;
+    case SyncResult::NotYetStarted:
+        folderMessage = tr( "The folder waits to start syncing." );
+        break;
+    case SyncResult::Unavailable:
+        folderMessage = tr( "Server is currently not available." );
+        break;
+    case SyncResult::SyncRunning:
+        folderMessage = tr("Sync is running.");
+        break;
+    case SyncResult::Success:
+        folderMessage = tr("Last Sync was successful.");
+        break;
+    case SyncResult::Error:
+        folderMessage = tr( "Syncing Error." );
+        break;
+    case SyncResult::SetupError:
+        folderMessage = tr( "Setup Error." );
+        break;
+    default:
+        folderMessage = tr( "Undefined Error State." );
+    }
+
+    _folderMessage = folderMessage;
+    _lastSyncTime = result.syncTime();
+
+    if( result.errorStrings().count() ) {
+        _errorLabel->setVisible(true);
+        _errorLabel->setTextFormat(Qt::RichText);
+        QString errStr;
+        foreach( QString err, result.errorStrings() ) {
+            errStr.append(QString("<p>%1</p>").arg(err));
+        }
+
+        _errorLabel->setText(errStr);
+    } else {
+        _errorLabel->setText(QString::null);
+        _errorLabel->setVisible(false);
+    }
+
+    slotSetFolderMessage();
+    if( syncStatus == SyncResult::SyncRunning ) {
+        _timer.stop();
+    } else {
+        _timer.start();
+    }
+
+    setSyncFileItems( result.syncFileItemVector() );
+
+}
+
+void FileItemDialog::slotSetFolderMessage()
+{
+    QDateTime now = QDateTime::currentDateTime();
+    int secs = _lastSyncTime.secsTo(now);
+
+    _timelabel->setText(tr("%1  (finished %2 sec. ago)").arg(_folderMessage).arg(secs));
+}
+
+void FileItemDialog::copyToClipboard()
+{
+    QString text;
+    QTextStream ts(&text);
+
+    int topLevelItems = _treeWidget->topLevelItemCount();
+    for (int i = 0; i < topLevelItems; i++) {
+        QTreeWidgetItem *item = _treeWidget->topLevelItem(i);
+        ts << left << qSetFieldWidth(50)
+           << item->data(0, Qt::DisplayRole).toString()
+           << right << qSetFieldWidth(6)
+           << item->data(1, Qt::DisplayRole).toString()
+           << endl;
+        int childItems = item->childCount();
+        for (int j = 0; j < childItems; j++) {
+            QTreeWidgetItem *child =item->child(j);
+            ts << left << qSetFieldWidth(0) << QLatin1String("   ")
+               << child->data(0,Qt::DisplayRole).toString()
+               << QString::fromLatin1(" (%1)").arg(
+                      child->data(1, Qt::DisplayRole).toString()
+                      )
+               << endl;
+        }
+    }
+
+    QApplication::clipboard()->setText(text);
+}
+
+void FileItemDialog::accept()
+{
+    _timer.stop();
+    QDialog::accept();
+}
+
+void FileItemDialog::setSyncFileItems( const SyncFileItemVector& list )
+{
+    _treeWidget->clear();
+    QStringList strings;
+    QFont headerFont;
+    headerFont.setWeight(QFont::Bold);
+
+    strings.clear();
+    strings.append(tr("Synced Files"));
+    _syncedFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_SUCCESS );
+    _syncedFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_syncedFileItem);
+
+    strings.clear();
+    strings.append(tr("New Files"));
+    _newFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_NEW );
+    _newFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_newFileItem);
+
+    strings.clear();
+    strings.append(tr("Deleted Files"));
+    _deletedFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_DELETED );
+    _deletedFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_deletedFileItem);
+
+    strings.clear();
+    strings.append(tr("Renamed Files"));
+    _renamedFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_RENAME);
+    _renamedFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_renamedFileItem);
+
+    strings.clear();
+    strings.append(tr("Ignored Files"));
+    _ignoredFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_IGNORE);
+    _ignoredFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_renamedFileItem);
+
+    strings.clear();
+    strings.append(tr("Errors"));
+    _errorFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_ERROR );
+    _errorFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_errorFileItem);
+
+    strings.clear();
+    strings.append(tr("Conflicts"));
+    _conflictFileItem = new QTreeWidgetItem( _treeWidget, strings, TYPE_CONFLICT);
+    _conflictFileItem->setChildIndicatorPolicy(QTreeWidgetItem::DontShowIndicatorWhenChildless);
+    _treeWidget->addTopLevelItem(_conflictFileItem);
+
+    QList<QTreeWidgetItem*> syncedItems;
+    QList<QTreeWidgetItem*> renamedItems;
+    QList<QTreeWidgetItem*> newItems;
+    QList<QTreeWidgetItem*> deletedItems;
+    QList<QTreeWidgetItem*> ignoredItems;
+    QList<QTreeWidgetItem*> conflictItems;
+    QList<QTreeWidgetItem*> errorItems;
+
+    quint64 overall_files = 0;
+
+    foreach( SyncFileItem item, list ) {
+        overall_files++;
+
+        QString dir;
+        QStringList str( item._file );
+        if( item._dir == SyncFileItem::Up )   dir = tr("Up");
+        if( item._dir == SyncFileItem::Down ) dir = tr("Down");
+        str << dir;
+
+        switch( item._instruction ) {
+        case   CSYNC_INSTRUCTION_NONE:
+            // do nothing.
+            break;
+        case CSYNC_INSTRUCTION_EVAL:
+            // should not happen
+            break;
+        case CSYNC_INSTRUCTION_REMOVE:
+        case CSYNC_INSTRUCTION_DELETED:
+            deletedItems.append( new QTreeWidgetItem(_deletedFileItem, str, FILE_TYPE) );
+            break;
+        case CSYNC_INSTRUCTION_RENAME:
+            renamedItems.append( new QTreeWidgetItem(_renamedFileItem, str, FILE_TYPE) );
+            break;
+        case CSYNC_INSTRUCTION_NEW:
+            newItems.append( new QTreeWidgetItem(_newFileItem, str, FILE_TYPE) );
+            break;
+        case CSYNC_INSTRUCTION_CONFLICT:
+            conflictItems.append( new QTreeWidgetItem(_conflictFileItem, str, FILE_TYPE) );
+            break;
+        case CSYNC_INSTRUCTION_IGNORE:
+            ignoredItems.append( new QTreeWidgetItem(_ignoredFileItem, str, FILE_TYPE) );
+            break;
+        case CSYNC_INSTRUCTION_SYNC:
+        case CSYNC_INSTRUCTION_UPDATED:
+            syncedItems.append( new QTreeWidgetItem(_syncedFileItem, str, FILE_TYPE) );
+            break;
+        case CSYNC_INSTRUCTION_STAT_ERROR:
+        case CSYNC_INSTRUCTION_ERROR:
+            errorItems.append( new QTreeWidgetItem(_errorFileItem, str, FILE_TYPE) );
+            break;
+        default:
+            break;
+        }
+    }
+
+    formatHeaderItem( _syncedFileItem, syncedItems );
+    formatHeaderItem( _newFileItem, newItems );
+    formatHeaderItem( _deletedFileItem, deletedItems );
+    formatHeaderItem( _renamedFileItem, renamedItems );
+    formatHeaderItem( _errorFileItem, errorItems );
+    formatHeaderItem( _conflictFileItem, conflictItems );
+    formatHeaderItem( _ignoredFileItem, ignoredItems );
+
+}
+
+void FileItemDialog::formatHeaderItem( QTreeWidgetItem *header, const QList<QTreeWidgetItem*>& list )
+{
+    if( !header ) return;
+
+    header->addChildren( list );
+    int count = list.count();
+#if LEAVE_THAT_TO_DESIGNERS
+    QColor col("#adc5d3");
+    header->setBackgroundColor(0, col);
+    header->setBackgroundColor(1, col);
+#endif
+    header->setText(1, QString::number( count ));
+    if( count ) {
+        QFont font;
+        font.setWeight( QFont::Bold );
+        header->setFont(0, font);
+        header->setFont(1, font);
+        header->setExpanded(true);
+    } else {
+        header->setExpanded(false);
+    }
+}
+
+}

src/mirall/fileitemdialog.h

@@ -0,0 +1,65 @@
+/*
+ * Copyright (C) by Klaas Freitag <freitag@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#ifndef FILEITEMDIALOG_H
+#define FILEITEMDIALOG_H
+
+#include <QDialog>
+#include <QDateTime>
+#include <QTimer>
+
+#include "mirall/syncfileitem.h"
+
+#include "ui_fileitemdialog.h"
+
+namespace Mirall {
+class Theme;
+class SyncResult;
+
+
+class FileItemDialog : public QDialog, public Ui::_fileItemDialog
+{
+    Q_OBJECT
+public:
+    explicit FileItemDialog(Theme*, QWidget *parent = 0);
+    void setSyncResult( const SyncResult& );
+
+signals:
+
+public slots:
+    void accept();
+
+protected slots:
+    void slotSetFolderMessage();
+    void copyToClipboard();
+
+private:
+    void setSyncFileItems( const SyncFileItemVector& list );
+    void formatHeaderItem( QTreeWidgetItem *, const QList<QTreeWidgetItem*>& );
+
+    QTreeWidgetItem *_newFileItem;
+    QTreeWidgetItem *_syncedFileItem;
+    QTreeWidgetItem *_deletedFileItem;
+    QTreeWidgetItem *_renamedFileItem;
+    QTreeWidgetItem *_errorFileItem;
+    QTreeWidgetItem *_conflictFileItem;
+    QTreeWidgetItem *_ignoredFileItem;
+
+    Theme    *_theme;
+    QString   _folderMessage;
+    QDateTime _lastSyncTime;
+    QTimer    _timer;
+};
+
+}
+#endif // FILEITEMDIALOG_H

src/mirall/fileitemdialog.ui

@@ -0,0 +1,87 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ui version="4.0">
+ <class>_fileItemDialog</class>
+ <widget class="QWidget" name="_fileItemDialog">
+  <property name="geometry">
+   <rect>
+    <x>0</x>
+    <y>0</y>
+    <width>612</width>
+    <height>543</height>
+   </rect>
+  </property>
+  <property name="windowTitle">
+   <string>Form</string>
+  </property>
+  <layout class="QGridLayout" name="gridLayout">
+   <item row="0" column="0">
+    <layout class="QVBoxLayout" name="verticalLayout">
+     <item>
+      <widget class="QLabel" name="label">
+       <property name="font">
+        <font>
+         <pointsize>14</pointsize>
+        </font>
+       </property>
+       <property name="text">
+        <string>Detailed Sync Protocol</string>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QTreeWidget" name="_treeWidget">
+       <property name="alternatingRowColors">
+        <bool>true</bool>
+       </property>
+       <property name="rootIsDecorated">
+        <bool>true</bool>
+       </property>
+       <property name="uniformRowHeights">
+        <bool>true</bool>
+       </property>
+       <property name="sortingEnabled">
+        <bool>true</bool>
+       </property>
+       <property name="columnCount">
+        <number>2</number>
+       </property>
+       <column>
+        <property name="text">
+         <string notr="true">1</string>
+        </property>
+       </column>
+       <column>
+        <property name="text">
+         <string notr="true">2</string>
+        </property>
+       </column>
+      </widget>
+     </item>
+     <item>
+      <widget class="QLabel" name="_timelabel">
+       <property name="text">
+        <string>TextLabel</string>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QLabel" name="_errorLabel">
+       <property name="text">
+        <string>TextLabel</string>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QDialogButtonBox" name="_dialogButtonBox">
+       <property name="standardButtons">
+        <set>QDialogButtonBox::Close</set>
+       </property>
+      </widget>
+     </item>
+    </layout>
+   </item>
+  </layout>
+ </widget>
+ <resources/>
+ <connections/>
+</ui>

src/mirall/folder.cpp

@@ -48,7 +48,6 @@ Folder::Folder(const QString &alias, const QString &path, const QString& secondP
     QObject::connect(_pollTimer, SIGNAL(timeout()), this, SLOT(slotPollTimerTimeout()));
     _pollTimer->start();
 
-#ifdef USE_INOTIFY
     _watcher = new Mirall::FolderWatcher(path, this);
 
     MirallConfigFile cfg;
@@ -57,7 +56,6 @@ Folder::Folder(const QString &alias, const QString &path, const QString& secondP
 
     QObject::connect(_watcher, SIGNAL(folderChanged(const QStringList &)),
                      SLOT(slotChanged(const QStringList &)));
-#endif
     QObject::connect(this, SIGNAL(syncStarted()),
                      SLOT(slotSyncStarted()));
     QObject::connect(this, SIGNAL(syncFinished(const SyncResult &)),
@@ -153,9 +151,7 @@ bool Folder::syncEnabled() const
 void Folder::setSyncEnabled( bool doit )
 {
   _enabled = doit;
-#ifdef USE_INOTIFY
   _watcher->setEventsEnabled( doit );
-#endif
   if( doit && ! _pollTimer->isActive() ) {
       _pollTimer->start();
   }
@@ -217,12 +213,10 @@ void Folder::incrementErrorCount()
   // of the watcher is doubled.
   _errorCount++;
   if( _errorCount > 1 ) {
-#ifdef USE_INOTIFY
     int interval = _watcher->eventInterval();
     int newInt = 2*interval;
     qDebug() << "Set new watcher interval to " << newInt;
     _watcher->setEventInterval( newInt );
-#endif
     _errorCount = 0;
   }
 }
@@ -262,9 +256,7 @@ void Folder::startSync( const QStringList &pathList )
 void Folder::slotPollTimerTimeout()
 {
     qDebug() << "* Polling" << alias() << "for changes. Ignoring all pending events until now";
-#ifdef USE_INOTIFY
     _watcher->clearPendingEvents();
-#endif
     evaluateSync(QStringList());
 }
 
@@ -283,16 +275,12 @@ void Folder::slotChanged(const QStringList &pathList)
 void Folder::slotSyncStarted()
 {
     // disable events until syncing is done
-#ifdef USE_INOTIFY
     _watcher->setEventsEnabled(false);
-#endif
 }
 
 void Folder::slotSyncFinished(const SyncResult &result)
 {
-#ifdef USE_INOTIFY
-    _watcher->setEventsEnabled(true);
-#endif
+    _watcher->setEventsEnabledDelayed(2000);
 
     qDebug() << "OO folder slotSyncFinished: result: " << int(result.status()) << " local: " << result.localRunOnly();
     emit syncStateChange();

src/mirall/folder.h

@@ -21,6 +21,7 @@
 #include <QString>
 #include <QStringList>
 #include <QHash>
+#include <QTimer>
 
 #if QT_VERSION >= 0x040700
 #include <QNetworkConfigurationManager>
@@ -29,15 +30,12 @@
 #include "mirall/syncresult.h"
 
 class QAction;
-class QTimer;
 class QIcon;
 class QFileSystemWatcher;
 
 namespace Mirall {
 
-#ifdef USE_INOTIFY
 class FolderWatcher;
-#endif
 
 class Folder : public QObject
 {
@@ -184,9 +182,7 @@ signals:
     void scheduleToSync( const QString& );
 
 protected:
-#ifdef USE_INOTIFY
     FolderWatcher *_watcher;
-#endif
     int _errorCount;
     SyncResult _syncResult;
 

src/mirall/folderman.cpp

@@ -21,6 +21,13 @@
 #include "mirall/inotify.h"
 #include "mirall/theme.h"
 
+#ifdef Q_OS_MAC
+#include <CoreServices/CoreServices.h>
+#endif
+#ifdef Q_OS_WIN
+#include <shlobj.h>
+#endif
+
 #include <QDesktopServices>
 #include <QtCore>
 
@@ -145,6 +152,38 @@ QString FolderMan::unescapeAlias( const QString& alias ) const
     return a;
 }
 
+void FolderMan::setupFavLink(const QString &folder)
+{
+#ifdef Q_OS_WIN
+    // Windows Explorer: Place under "Favorites" (Links)
+    wchar_t path[MAX_PATH];
+    SHGetSpecialFolderPath(0, path, CSIDL_PROFILE, FALSE);
+    QString profile =  QDir::fromNativeSeparators(QString::fromWCharArray(path));
+    QDir folderDir(QDir::fromNativeSeparators(folder));
+    QString linkName = profile+QLatin1String("/Links/") + folderDir.dirName() + QLatin1String(".lnk");
+    if (!QFile::link(folder, linkName))
+        qDebug() << Q_FUNC_INFO << "linking" << folder << "to" << linkName << "failed!";
+#elif defined (Q_OS_MAC)
+    // Finder: Place under "Places"
+    QString folderUrl = QUrl::fromLocalFile(folder).toString();
+    CFStringRef folderCFStr = CFStringCreateWithCharacters(0, reinterpret_cast<const UniChar *>(folderUrl.unicode()),
+                                                   folder.length());
+    CFURLRef urlRef = CFURLCreateWithString(NULL, folderCFStr, 0);
+    LSSharedFileListRef placesItems = LSSharedFileListCreate(0, kLSSharedFileListFavoriteItems, 0);
+    if (placesItems) {
+        //Insert an item to the list.
+        LSSharedFileListItemRef item = LSSharedFileListInsertItemURL(placesItems,
+                                                                     kLSSharedFileListItemBeforeFirst, 0, 0,
+                                                                     urlRef, 0, 0);
+        if (item)
+            CFRelease(item);
+    }
+    CFRelease(placesItems);
+    CFRelease(folderCFStr);
+    CFRelease(urlRef);
+#endif
+}
+
 // filename is the name of the file only, it does not include
 // the configuration directory path
 Folder* FolderMan::setupFolderFromConfigFile(const QString &file) {
@@ -389,6 +428,7 @@ void FolderMan::addFolderDefinition( const QString& backend, const QString& alia
     settings.setValue(QString::fromLatin1("%1/onlyThisLAN").arg(escapedAlias), onlyThisLAN );
     settings.sync();
 
+    setupFavLink(sourceFolder);
 }
 
 void FolderMan::removeAllFolderDefinitions()

src/mirall/folderman.h

@@ -21,6 +21,7 @@
 
 #include "mirall/folder.h"
 #include "mirall/folderwatcher.h"
+#include "mirall/syncfileitem.h"
 
 class QSignalMapper;
 
@@ -101,6 +102,7 @@ private:
     // finds all folder configuration files
     // and create the folders
     int setupKnownFolders();
+    void setupFavLink(const QString& folder);
 
     // Escaping of the alias which is used in QSettings AND the file
     // system, thus need to be escaped.

src/mirall/folderwatcher.cpp

@@ -28,20 +28,18 @@
 #include <QStringList>
 #include <QTimer>
 
-#ifdef USE_INOTIFY
-#include <sys/inotify.h>
-#endif
-
-static const uint32_t standard_event_mask =
-#ifdef USE_INOTIFY
-    IN_CLOSE_WRITE | IN_ATTRIB | IN_MOVE | IN_CREATE | IN_DELETE | IN_DELETE_SELF | IN_MOVE_SELF | IN_UNMOUNT | IN_ONLYDIR | IN_DONT_FOLLOW;
-#else
-    0;
-#endif
 /* minimum amount of seconds between two
    events  to consider it a new event */
 #define DEFAULT_EVENT_INTERVAL_MSEC 1000
 
+#if defined(Q_OS_WIN)
+#include "mirall/folderwatcher_win.h"
+#elif defined(Q_OS_MAC)
+#include "mirall/folderwatcher_mac.h"
+#elif defined(USE_INOTIFY)
+#include "mirall/folderwatcher_inotify.h"
+#endif
+
 namespace Mirall {
 
 FolderWatcher::FolderWatcher(const QString &root, QObject *parent)
@@ -50,18 +48,13 @@ FolderWatcher::FolderWatcher(const QString &root, QObject *parent)
       _eventInterval(DEFAULT_EVENT_INTERVAL_MSEC),
       _root(root),
       _processTimer(new QTimer(this)),
-      _lastMask(0),
       _initialSyncDone(false)
 {
-#ifdef USE_INOTIFY
+    _d = new FolderWatcherPrivate(this);
+
     _processTimer->setSingleShot(true);
     QObject::connect(_processTimer, SIGNAL(timeout()), this, SLOT(slotProcessTimerTimeout()));
 
-    _inotify = new INotify(standard_event_mask);
-    slotAddFolderRecursive(root);
-    QObject::connect(_inotify, SIGNAL(notifyEvent(int, int, const QString &)),
-                     this, SLOT(slotINotifyEvent(int, int, const QString &)));
-#endif
     // do a first synchronization to get changes while
     // the application was not running
     setProcessTimer();
@@ -69,7 +62,7 @@ FolderWatcher::FolderWatcher(const QString &root, QObject *parent)
 
 FolderWatcher::~FolderWatcher()
 {
-
+    delete _d;
 }
 
 QString FolderWatcher::root() const
@@ -99,11 +92,22 @@ void FolderWatcher::addIgnore(const QString &pattern)
     _ignores.append(pattern);
 }
 
+QStringList FolderWatcher::ignores() const
+{
+    return _ignores;
+}
+
 bool FolderWatcher::eventsEnabled() const
 {
     return _eventsEnabled;
 }
 
+void FolderWatcher::setEventsEnabledDelayed( int delay_msec )
+{
+    qDebug() << "Starting Event logging again in " << delay_msec << " milliseconds";
+    QTimer::singleShot( delay_msec, this, SLOT(setEventsEnabled()));
+}
+
 void FolderWatcher::setEventsEnabled(bool enabled)
 {
     qDebug() << "    * event notification " << (enabled ? "enabled" : "disabled");
@@ -139,131 +143,6 @@ void FolderWatcher::setEventInterval(int seconds)
     _eventInterval = seconds;
 }
 
-QStringList FolderWatcher::folders() const
-{
-#ifdef USE_INOTIFY
-    return _inotify->directories();
-#else
-    return QStringList();
-#endif
-}
-
-void FolderWatcher::slotAddFolderRecursive(const QString &path)
-{
-    int subdirs = 0;
-    qDebug() << "(+) Watcher:" << path;
-#ifdef USE_INOTIFY
-
-    _inotify->addPath(path);
-    QStringList watchedFolders(_inotify->directories());
-    // qDebug() << "currently watching " << watchedFolders;
-    QStringListIterator subfoldersIt(FileUtils::subFoldersList(path, FileUtils::SubFolderRecursive));
-    while (subfoldersIt.hasNext()) {
-        QString subfolder = subfoldersIt.next();
-        // qDebug() << "  (**) subfolder: " << subfolder;
-        QDir folder (subfolder);
-        if (folder.exists() && !watchedFolders.contains(folder.path())) {
-            subdirs++;
-            // check that it does not match the ignore list
-            foreach ( const QString& pattern, _ignores) {
-                QRegExp regexp(pattern);
-                regexp.setPatternSyntax(QRegExp::Wildcard);
-                if ( regexp.exactMatch(folder.path()) ) {
-                    qDebug() << "* Not adding" << folder.path();
-                    continue;
-                }
-
-            }
-            _inotify->addPath(folder.path());
-        }
-        else
-            qDebug() << "    `-> discarded:" << folder.path();
-    }
-    if (subdirs >0)
-        qDebug() << "    `-> and" << subdirs << "subdirectories";
-#else
-    qDebug() << "** Watcher is not compiled in!";
-#endif
-}
-
-void FolderWatcher::slotINotifyEvent(int mask, int cookie, const QString &path)
-{
-    int lastMask = _lastMask;
-    QString lastPath = _lastPath;
-
-    _lastMask = mask;
-    _lastPath = path;
-
-    if( ! eventsEnabled() ) return;
-#ifdef USE_INOTIFY
-    qDebug() << "** Inotify Event " << mask << " on " << path;
-    // cancel close write events that come after create
-    if (lastMask == IN_CREATE && mask == IN_CLOSE_WRITE
-        && lastPath == path ) {
-        return;
-    }
-
-    if (IN_IGNORED & mask) {
-        //qDebug() << "IGNORE event";
-        return;
-    }
-
-    if (IN_Q_OVERFLOW & mask) {
-        //qDebug() << "OVERFLOW";
-    }
-
-    if (mask & IN_CREATE) {
-        //qDebug() << cookie << " CREATE: " << path;
-        if (QFileInfo(path).isDir()) {
-            //setEventsEnabled(false);
-            slotAddFolderRecursive(path);
-            //setEventsEnabled(true);
-        }
-    }
-    else if (mask & IN_DELETE) {
-        //qDebug() << cookie << " DELETE: " << path;
-        if ( QFileInfo(path).isDir() && _inotify->directories().contains(path) ) {
-            qDebug() << "(-) Watcher:" << path;
-            _inotify->removePath(path);
-        }
-    }
-    else if (mask & IN_CLOSE_WRITE) {
-        //qDebug() << cookie << " WRITABLE CLOSED: " << path;
-    }
-    else if (mask & IN_MOVE) {
-        //qDebug() << cookie << " MOVE: " << path;
-    }
-    else {
-        //qDebug() << cookie << " OTHER " << mask << " :" << path;
-    }
-
-    foreach (const QString& pattern, _ignores) {
-        QRegExp regexp(pattern);
-        regexp.setPatternSyntax(QRegExp::Wildcard);
-
-        if (regexp.exactMatch(path)) {
-            qDebug() << "* Discarded by ignore pattern: " << path;
-            return;
-        }
-        QFileInfo fInfo(path);
-        if( regexp.exactMatch(fInfo.fileName())) {
-            qDebug() << "* Discarded by ignore pattern:" << path;
-            return;
-        }
-        if( fInfo.isHidden() ) {
-            qDebug() << "* Discarded as is hidden!";
-            return;
-        }
-    }
-
-    if( !_pendingPathes.contains( path )) {
-        _pendingPathes[path] = 0;
-    }
-    _pendingPathes[path] = _pendingPathes[path]+mask;
-#endif
-    setProcessTimer();
-}
-
 void FolderWatcher::slotProcessTimerTimeout()
 {
     qDebug() << "* Processing of event queue for" << root();
@@ -272,7 +151,7 @@ void FolderWatcher::slotProcessTimerTimeout()
         QStringList notifyPaths = _pendingPathes.keys();
         _pendingPathes.clear();
         //qDebug() << lastEventTime << eventTime;
-        qDebug() << "  * Notify" << notifyPaths.size() << "changed items for" << root();
+        qDebug() << "  * Notify" << notifyPaths.size() << "change items for" << root();
         emit folderChanged(notifyPaths);
         _initialSyncDone = true;
     }
@@ -283,12 +162,23 @@ void FolderWatcher::setProcessTimer()
     if (!_processTimer->isActive()) {
         qDebug() << "* Pending events for" << root()
                  << "will be processed after events stop for"
-                 << eventInterval() << "seconds ("
+                 << eventInterval() << "milliseconds ("
                  << QTime::currentTime().addSecs(eventInterval()).toString(QLatin1String("HH:mm:ss"))
                  << ")." << _pendingPathes.size() << "events until now )";
     }
     _processTimer->start(eventInterval());
 }
 
+void FolderWatcher::changeDetected(const QString& f)
+{
+    if( ! eventsEnabled() ) {
+        qDebug() << "FolderWatcher::changeDetected when eventsEnabled() -> ignore";
+        return;
+    }
+
+    _pendingPathes[f] = 1; //_pendingPathes[path]+mask;
+    setProcessTimer();
 }
 
+} // namespace Mirall
+

src/mirall/folderwatcher.h

@@ -30,9 +30,7 @@ class QTimer;
 
 namespace Mirall {
 
-#ifdef USE_INOTIFY
-class INotify;
-#endif
+class FolderWatcherPrivate;
 
 /**
  * Watches a folder and sub folders for changes
@@ -53,11 +51,6 @@ public:
     FolderWatcher(const QString &root, QObject *parent = 0L);
     ~FolderWatcher();
 
-    /**
-     * All watched folders and subfolders
-     */
-    QStringList folders() const;
-
     /**
      * Root path being monitored
      */
@@ -82,13 +75,6 @@ public:
      */
     bool eventsEnabled() const;
 
-    /**
-     * Enabled or disables folderChanged() events.
-     * If disabled, events are accumulated and emptied
-     * the next time a folderChanged() event happens.
-     */
-    void setEventsEnabled(bool enabled);
-
     /**
      * Clear all pending events
      */
@@ -106,6 +92,22 @@ public:
      */
     void setEventInterval(int seconds);
 
+    QStringList ignores() const;
+public slots:
+    /**
+     * Enabled or disables folderChanged() events.
+     * If disabled, events are accumulated and emptied
+     * the next time a folderChanged() event happens.
+     */
+    void setEventsEnabled(bool enabled=true);
+
+    /**
+     * @brief setEventsEnabledDelayed - start event logging after a while
+     * @param delay     - delay time in milliseconds
+     * @param enabled   - enable the events.
+     */
+    void setEventsEnabledDelayed( int );
+
 signals:
     /**
      * Emitted when one of the paths is changed
@@ -116,32 +118,27 @@ protected:
     void setProcessTimer();
 
 protected slots:
-    void slotINotifyEvent(int mask, int cookie, const QString &path);
-    void slotAddFolderRecursive(const QString &path);
     // called when the manually process timer triggers
     void slotProcessTimerTimeout();
+    void changeDetected(const QString &f);
+
+protected:
+    QHash<QString, int> _pendingPathes;
 
 private:
     bool _eventsEnabled;
     int _eventInterval;
-#ifdef USE_INOTIFY
-    INotify *_inotify;
-#endif
+    FolderWatcherPrivate *_d;
     QString _root;
     // paths pending to notified
     // QStringList _pendingPaths;
-    QHash<QString, int> _pendingPathes;
-
     QTimer *_processTimer;
-
-    // to cancel events that belong to the same action
-    int _lastMask;
-    QString _lastPath;
     QStringList _ignores;
-
     // for the initial synchronization, without
     // any file changed
     bool _initialSyncDone;
+
+    friend class FolderWatcherPrivate;
 };
 
 }

src/mirall/folderwatcher_inotify.cpp

@@ -0,0 +1,158 @@
+/*
+ * Copyright (C) by Duncan Mac-Vicar P. <duncan@kde.org>
+ * Copyright (C) by Daniel Molkentin <danimo@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#include <sys/inotify.h>
+
+#include "mirall/inotify.h"
+#include "mirall/folderwatcher.h"
+#include "mirall/fileutils.h"
+
+#include "mirall/folderwatcher_inotify.h"
+
+#include <QDir>
+#include <QFileInfo>
+#include <QDebug>
+
+namespace Mirall {
+
+static const uint32_t standard_event_mask =
+    IN_CLOSE_WRITE | IN_ATTRIB | IN_MOVE |
+    IN_CREATE |IN_DELETE | IN_DELETE_SELF |
+    IN_MOVE_SELF |IN_UNMOUNT |IN_ONLYDIR |
+    IN_DONT_FOLLOW;
+
+FolderWatcherPrivate::FolderWatcherPrivate(FolderWatcher *p)
+    : QObject(), _parent(p), _lastMask(0)
+
+{
+    _inotify = new INotify(this, standard_event_mask);
+    slotAddFolderRecursive(_parent->root());
+    QObject::connect(_inotify, SIGNAL(notifyEvent(int, int, const QString &)),
+                     this, SLOT(slotINotifyEvent(int, int, const QString &)));
+
+}
+
+void FolderWatcherPrivate::slotAddFolderRecursive(const QString &path)
+{
+    int subdirs = 0;
+    qDebug() << "(+) Watcher:" << path;
+
+    _inotify->addPath(path);
+    QStringList watchedFolders(_inotify->directories());
+    // qDebug() << "currently watching " << watchedFolders;
+    QStringListIterator subfoldersIt(FileUtils::subFoldersList(path, FileUtils::SubFolderRecursive));
+    while (subfoldersIt.hasNext()) {
+        QString subfolder = subfoldersIt.next();
+        // qDebug() << "  (**) subfolder: " << subfolder;
+        QDir folder (subfolder);
+        if (folder.exists() && !watchedFolders.contains(folder.path())) {
+            subdirs++;
+            // check that it does not match the ignore list
+            foreach ( const QString& pattern, _parent->ignores()) {
+                QRegExp regexp(pattern);
+                regexp.setPatternSyntax(QRegExp::Wildcard);
+                if ( regexp.exactMatch(folder.path()) ) {
+                    qDebug() << "* Not adding" << folder.path();
+                    continue;
+                }
+
+            }
+            _inotify->addPath(folder.path());
+        }
+        else
+            qDebug() << "    `-> discarded:" << folder.path();
+    }
+    if (subdirs >0)
+        qDebug() << "    `-> and" << subdirs << "subdirectories";
+}
+
+void FolderWatcherPrivate::slotINotifyEvent(int mask, int cookie, const QString &path)
+{
+    int lastMask = _lastMask;
+    QString lastPath = _lastPath;
+
+    _lastMask = mask;
+    _lastPath = path;
+
+    // TODO: Unify behaviour acress backends!
+    if( ! _parent->eventsEnabled() ) return;
+    qDebug() << "** Inotify Event " << mask << " on " << path;
+    // cancel close write events that come after create
+    if (lastMask == IN_CREATE && mask == IN_CLOSE_WRITE
+        && lastPath == path ) {
+        return;
+    }
+
+    if (IN_IGNORED & mask) {
+        //qDebug() << "IGNORE event";
+        return;
+    }
+
+    if (IN_Q_OVERFLOW & mask) {
+        //qDebug() << "OVERFLOW";
+    }
+
+    if (mask & IN_CREATE) {
+        //qDebug() << cookie << " CREATE: " << path;
+        if (QFileInfo(path).isDir()) {
+            //setEventsEnabled(false);
+            slotAddFolderRecursive(path);
+            //setEventsEnabled(true);
+        }
+    }
+    else if (mask & IN_DELETE) {
+        //qDebug() << cookie << " DELETE: " << path;
+        if ( QFileInfo(path).isDir() && _inotify->directories().contains(path) ) {
+            qDebug() << "(-) Watcher:" << path;
+            _inotify->removePath(path);
+        }
+    }
+    else if (mask & IN_CLOSE_WRITE) {
+        //qDebug() << cookie << " WRITABLE CLOSED: " << path;
+    }
+    else if (mask & IN_MOVE) {
+        //qDebug() << cookie << " MOVE: " << path;
+    }
+    else {
+        //qDebug() << cookie << " OTHER " << mask << " :" << path;
+    }
+
+    foreach (const QString& pattern, _parent->ignores()) {
+        QRegExp regexp(pattern);
+        regexp.setPatternSyntax(QRegExp::Wildcard);
+
+        if (regexp.exactMatch(path)) {
+            qDebug() << "* Discarded by ignore pattern: " << path;
+            return;
+        }
+        QFileInfo fInfo(path);
+        if( regexp.exactMatch(fInfo.fileName())) {
+            qDebug() << "* Discarded by ignore pattern:" << path;
+            return;
+        }
+        if( fInfo.isHidden() ) {
+            qDebug() << "* Discarded as is hidden!";
+            return;
+        }
+    }
+
+    if( !_parent->_pendingPathes.contains( path )) {
+        _parent->_pendingPathes[path] = 0;
+    }
+    _parent->_pendingPathes[path] = _parent->_pendingPathes[path]+mask;
+    _parent->setProcessTimer();
+}
+
+} // namespace Mirall

src/mirall/folderwatcher_inotify.h

@@ -0,0 +1,42 @@
+/*
+ * Copyright (C) by Daniel Molkentin <danimo@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#ifndef MIRALL_FOLDERWATCHER_INOTIFY_H
+#define MIRALL_FOLDERWATCHER_INOTIFY_H
+
+#include <QObject>
+
+namespace Mirall {
+
+class INotify;
+class FolderWatcher;
+
+class FolderWatcherPrivate : public QObject {
+    Q_OBJECT
+public:
+    FolderWatcherPrivate(FolderWatcher *p);
+private slots:
+    void slotAddFolderRecursive(const QString &path);
+    void slotINotifyEvent(int mask, int cookie, const QString &path);
+private:
+    INotify *_inotify;
+    FolderWatcher *_parent;
+    // to cancel events that belong to the same action
+    int _lastMask;
+    QString _lastPath;
+};
+
+}
+
+#endif // MIRALL_FOLDERWATCHER_INOTIFY_H

src/mirall/folderwatcher_mac.cpp

@@ -0,0 +1,86 @@
+/*
+ * Copyright (C) by Markus Goetz <markus@woboq.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+#include "config.h"
+
+#include "mirall/folder.h"
+#include "mirall/folderwatcher.h"
+#include "mirall/folderwatcher_mac.h"
+
+
+#include <cerrno>
+#include <QDebug>
+#include <QStringList>
+
+
+
+namespace Mirall {
+
+FolderWatcherPrivate::FolderWatcherPrivate(FolderWatcher *p)
+    : parent(p)
+{
+    folder = parent->root();
+    this->startWatching();
+}
+
+FolderWatcherPrivate::~FolderWatcherPrivate()
+{
+    FSEventStreamStop(stream);
+    FSEventStreamInvalidate(stream);
+}
+
+static void callback(
+        ConstFSEventStreamRef streamRef,
+        void *clientCallBackInfo,
+        size_t numEvents,
+        void *eventPaths,
+        const FSEventStreamEventFlags eventFlags[],
+        const FSEventStreamEventId eventIds[])
+{
+    qDebug() << "FolderWatcherPrivate::callback by OS X";
+    reinterpret_cast<FolderWatcherPrivate*>(clientCallBackInfo)->doNotifyParent();
+}
+
+void FolderWatcherPrivate::startWatching()
+{
+    qDebug() << "FolderWatcherPrivate::startWatching()" << folder;
+    CFStringRef folderCF = CFStringCreateWithCharacters(0, reinterpret_cast<const UniChar *>(folder.unicode()),
+                                                        folder.length());
+    CFArrayRef pathsToWatch = CFStringCreateArrayBySeparatingStrings (NULL, folderCF, CFSTR(":"));
+
+            //CFStringCreateArrayBySeparatingStrings (NULL, folderCF, CFSTR(":"));
+
+    FSEventStreamContext ctx =  {0, this, NULL, NULL, NULL};
+
+    // TODO: Add kFSEventStreamCreateFlagFileEvents ?
+
+    stream = FSEventStreamCreate(NULL,
+                                 &callback,
+                                 &ctx,
+                                 pathsToWatch,
+                                 kFSEventStreamEventIdSinceNow,
+                                 0, // latency
+                                 kFSEventStreamCreateFlagNone
+                                 );
+
+    FSEventStreamScheduleWithRunLoop(stream, CFRunLoopGetCurrent(), kCFRunLoopDefaultMode);
+    FSEventStreamStart(stream);
+}
+
+void FolderWatcherPrivate::doNotifyParent() {
+    parent->changeDetected(folder);
+}
+
+
+
+} // ns mirall

src/mirall/folderwatcher_mac.h

@@ -0,0 +1,47 @@
+/*
+ * Copyright (C) by Markus Goetz <markus@woboq.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#ifndef MIRALL_FOLDERWATCHER_MAC_H
+#define MIRALL_FOLDERWATCHER_MAC_H
+
+#include <QObject>
+#include <QString>
+
+#include <CoreServices/CoreServices.h>
+
+
+namespace Mirall
+{
+
+class FolderWatcherPrivate
+{
+public:
+
+    FolderWatcherPrivate(FolderWatcher *p);
+    ~FolderWatcherPrivate();
+
+    void startWatching();
+    void doNotifyParent();
+
+private:
+    FolderWatcher *parent;
+
+    QString folder;
+
+    FSEventStreamRef stream;
+};
+
+}
+
+#endif

src/mirall/folderwatcher_win.cpp

@@ -0,0 +1,91 @@
+/*
+ * Copyright (C) by Daniel Molkentin <danimo@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#include <QThread>
+#include <QDebug>
+
+#include "mirall/folderwatcher.h"
+#include "mirall/folderwatcher_win.h"
+
+#include <stdlib.h>
+#include <stdio.h>
+#include <tchar.h>
+
+namespace Mirall {
+
+void WatcherThread::run()
+{
+    _handle = FindFirstChangeNotification((wchar_t*)_path.utf16(),
+                                         true, // recursive watch
+                                         FILE_NOTIFY_CHANGE_FILE_NAME |
+                                         FILE_NOTIFY_CHANGE_DIR_NAME |
+                                         FILE_NOTIFY_CHANGE_LAST_WRITE);
+
+    if (_handle == INVALID_HANDLE_VALUE)
+    {
+        qDebug() << Q_FUNC_INFO << "FindFirstChangeNotification function failed, stopping watcher!";
+        FindCloseChangeNotification(_handle);
+        _handle = 0;
+        return;
+    }
+
+    if (_handle == NULL)
+    {
+        qDebug() << Q_FUNC_INFO << "FindFirstChangeNotification returned null, stopping watcher!";
+        FindCloseChangeNotification(_handle);
+        _handle = 0;
+        return;
+    }
+
+    while(true) {
+        switch(WaitForSingleObject(_handle, /*wait*/ INFINITE)) {
+        case WAIT_OBJECT_0:
+            if (FindNextChangeNotification(_handle) == false) {
+                qDebug() << Q_FUNC_INFO << "FindFirstChangeNotification returned FALSE, stopping watcher!";
+                FindCloseChangeNotification(_handle);
+                _handle = 0;
+                return;
+            }
+            qDebug() << Q_FUNC_INFO << "Change detected in" << _path << "from" << QThread::currentThread    ();
+            emit changed(_path);
+            break;
+        default:
+            qDebug()  << Q_FUNC_INFO << "Error while watching";
+        }
+    }
+}
+
+WatcherThread::~WatcherThread()
+{
+    if (_handle)
+        FindCloseChangeNotification(_handle);
+}
+
+FolderWatcherPrivate::FolderWatcherPrivate(FolderWatcher *p)
+    : _parent(p)
+{
+    _thread = new WatcherThread(p->root());
+    connect(_thread, SIGNAL(changed(const QString&)),
+            _parent,SLOT(changeDetected(const QString&)));
+    _thread->start();
+}
+
+FolderWatcherPrivate::~FolderWatcherPrivate()
+{
+    _thread->terminate();
+    _thread->wait();
+    delete _thread;
+}
+
+} // namespace Mirall

src/mirall/folderwatcher_win.h

@@ -0,0 +1,58 @@
+/*
+ * Copyright (C) by Daniel Molkentin <danimo@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#ifndef MIRALL_FOLDERWATCHER_WIN_H
+#define MIRALL_FOLDERWATCHER_WIN_H
+
+#include <QThread>
+#include <windows.h>
+
+namespace Mirall {
+
+class FolderWatcher;
+
+// watcher thread
+
+class WatcherThread : public QThread {
+    Q_OBJECT
+public:
+    WatcherThread(const QString &path) :
+        QThread(), _path(path), _handle(0) {}
+
+    ~WatcherThread();
+
+protected:
+    void run();
+
+signals:
+    void changed(const QString &path);
+
+private:
+    QString _path;
+    HANDLE _handle;
+};
+
+class FolderWatcherPrivate : public QObject {
+    Q_OBJECT
+public:
+    FolderWatcherPrivate(FolderWatcher *p);
+    ~FolderWatcherPrivate();
+private:
+    FolderWatcher *_parent;
+    WatcherThread *_thread;
+};
+
+}
+
+#endif // MIRALL_FOLDERWATCHER_WIN_H

src/mirall/folderwizard.cpp

@@ -78,7 +78,7 @@ bool FolderWizardSourcePage::isComplete() const
   if( ! map ) return false;
 
   if( isOk ) {
-    Folder::Map::const_iterator i = map->begin();
+    Folder::Map::const_iterator i = map->constBegin();
     while( isOk && i != map->constEnd() ) {
       Folder *f = static_cast<Folder*>(i.value());
       QString folderDir = QDir( f->path() ).canonicalPath();
@@ -110,7 +110,7 @@ bool FolderWizardSourcePage::isComplete() const
     isOk = false;
   }
 
-  Folder::Map::const_iterator i = map->begin();
+  Folder::Map::const_iterator i = map->constBegin();
   bool goon = true;
   while( goon && i != map->constEnd() ) {
     Folder *f = static_cast<Folder*>(i.value());

src/mirall/folderwizardtargetpage.ui

@@ -71,9 +71,6 @@
            <property name="text">
             <string/>
            </property>
-           <property name="pixmap">
-            <pixmap resource="../../mirall.qrc">:/mirall/resources/owncloud-icon-48.png</pixmap>
-           </property>
           </widget>
          </item>
          <item row="0" column="1">

src/mirall/inotify.cpp

@@ -16,9 +16,7 @@
  */
 #include "config.h"
 
-#ifdef USE_INOTIFY
 #include <sys/inotify.h>
-#endif
 
 #include "inotify.h"
 #include "mirall/folder.h"
@@ -36,7 +34,9 @@
 
 namespace Mirall {
 
-INotify::INotify(int mask) : _mask(mask)
+INotify::INotify(QObject *parent, int mask)
+    : QObject(parent),
+      _mask(mask)
 {
     _fd = inotify_init();
     _notifier = new QSocketNotifier(_fd, QSocketNotifier::Read);

src/mirall/inotify.h

@@ -35,7 +35,7 @@ class INotify : public QObject
 
 public:
 
-    INotify(int mask);
+    INotify(QObject *parent, int mask);
     ~INotify();
 
     static void initialize();

src/mirall/logbrowser.cpp

@@ -30,70 +30,10 @@
 #include <QDebug>
 
 #include "mirall/mirallconfigfile.h"
+#include "mirall/logger.h"
 
 namespace Mirall {
 
-Logger* Logger::_instance=0;
-
-Logger::Logger( QObject* parent)
-: QObject(parent),
-  _showTime(true)
-{
-
-}
-
-Logger *Logger::instance()
-{
-    if( !Logger::_instance ) Logger::_instance = new Logger;
-    return Logger::_instance;
-}
-
-void Logger::destroy()
-{
-    if( Logger::_instance ) {
-        delete Logger::_instance;
-        Logger::_instance = 0;
-    }
-}
-
-void Logger::log(Log log)
-{
-    QString msg;
-    if( _showTime ) {
-        msg = log.timeStamp.toString(QLatin1String("MM-dd hh:mm:ss:zzz")) + QLatin1Char(' ');
-    }
-
-    if( log.source == Log::CSync ) {
-        // msg += "csync - ";
-    } else {
-        // msg += "ownCloud - ";
-    }
-    msg += log.message;
-    // _logs.append(log);
-    // std::cout << qPrintable(log.message) << std::endl;
-    emit newLog(msg);
-}
-
-void Logger::csyncLog( const QString& message )
-{
-    Log log;
-    log.source = Log::CSync;
-    log.timeStamp = QDateTime::currentDateTime();
-    log.message = message;
-
-    Logger::instance()->log(log);
-}
-
-void Logger::mirallLog( const QString& message )
-{
-    Log log_;
-    log_.source = Log::Mirall;
-    log_.timeStamp = QDateTime::currentDateTime();
-    log_.message = message;
-
-    Logger::instance()->log( log_ );
-}
-
 // ==============================================================================
 
 LogWidget::LogWidget(QWidget *parent)
@@ -248,7 +188,6 @@ void LogBrowser::search( const QString& str )
 void LogBrowser::slotSave()
 {
     _saveBtn->setEnabled(false);
-    QCoreApplication::processEvents();
 
     QString saveFile = QFileDialog::getSaveFileName( this, tr("Save log file"), QDir::homePath() );
 

src/mirall/logbrowser.h

@@ -28,43 +28,6 @@
 
 namespace Mirall {
 
-struct Log{
-  typedef enum{
-    Mirall,
-    CSync
-  } Source;
-
-  QDateTime timeStamp;
-  Source source;
-  QString message;
-};
-
-class Logger : public QObject
-{
-  Q_OBJECT
-public:
-  void log(Log log);
-
-  static void csyncLog( const QString& message );
-  static void mirallLog( const QString& message );
-
-  const QList<Log>& logs() const {return _logs;}
-
-  static Logger* instance();
-  static void destroy();
-
-signals:
-  void newLog(const QString&);
-
-protected:
-  Logger(QObject* parent=0);
-  QList<Log> _logs;
-  bool       _showTime;
-  bool       _doLogging;
-
-  static Logger* _instance;
-};
-
 class LogWidget : public QPlainTextEdit 
 {
     Q_OBJECT

src/mirall/logger.cpp

@@ -0,0 +1,80 @@
+/*
+ * Copyright (C) by Klaas Freitag <freitag@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#include "mirall/logger.h"
+
+namespace Mirall {
+
+Logger* Logger::_instance=0;
+
+Logger::Logger( QObject* parent)
+: QObject(parent),
+  _showTime(true)
+{
+
+}
+
+Logger *Logger::instance()
+{
+    if( !Logger::_instance ) Logger::_instance = new Logger;
+    return Logger::_instance;
+}
+
+void Logger::destroy()
+{
+    if( Logger::_instance ) {
+        delete Logger::_instance;
+        Logger::_instance = 0;
+    }
+}
+
+void Logger::log(Log log)
+{
+    QString msg;
+    if( _showTime ) {
+        msg = log.timeStamp.toString(QLatin1String("MM-dd hh:mm:ss:zzz")) + QLatin1Char(' ');
+    }
+
+    if( log.source == Log::CSync ) {
+        // msg += "csync - ";
+    } else {
+        // msg += "ownCloud - ";
+    }
+    msg += log.message;
+    // _logs.append(log);
+    // std::cout << qPrintable(log.message) << std::endl;
+    emit newLog(msg);
+}
+
+void Logger::csyncLog( const QString& message )
+{
+    Log log;
+    log.source = Log::CSync;
+    log.timeStamp = QDateTime::currentDateTime();
+    log.message = message;
+
+    Logger::instance()->log(log);
+}
+
+void Logger::mirallLog( const QString& message )
+{
+    Log log_;
+    log_.source = Log::Mirall;
+    log_.timeStamp = QDateTime::currentDateTime();
+    log_.message = message;
+
+    Logger::instance()->log( log_ );
+}
+
+} // namespace Mirall

src/mirall/logger.h

@@ -0,0 +1,64 @@
+/*
+ * Copyright (C) by Klaas Freitag <freitag@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#ifndef LOGGER_H
+#define LOGGER_H
+
+#include <QObject>
+#include <QList>
+#include <QDateTime>
+
+namespace Mirall {
+
+struct Log{
+  typedef enum{
+    Mirall,
+    CSync
+  } Source;
+
+  QDateTime timeStamp;
+  Source source;
+  QString message;
+};
+
+class Logger : public QObject
+{
+  Q_OBJECT
+public:
+  void log(Log log);
+
+  static void csyncLog( const QString& message );
+  static void mirallLog( const QString& message );
+
+  const QList<Log>& logs() const {return _logs;}
+
+  static Logger* instance();
+  static void destroy();
+
+signals:
+  void newLog(const QString&);
+  void guiLog(const QString&, const QString&);
+
+protected:
+  Logger(QObject* parent=0);
+  QList<Log> _logs;
+  bool       _showTime;
+  bool       _doLogging;
+
+  static Logger* _instance;
+};
+
+} // namespace Mirall
+
+#endif // LOGGER_H

src/mirall/mirallconfigfile.cpp

@@ -23,8 +23,6 @@
 #include <QtGui>
 
 #define DEFAULT_REMOTE_POLL_INTERVAL 30000 // default remote poll time in milliseconds
-#define DEFAULT_LOCAL_POLL_INTERVAL  10000 // default local poll time in milliseconds
-#define DEFAULT_POLL_TIMER_EXEED     10
 
 #define CA_CERTS_KEY QLatin1String("CaCertificates")
 
@@ -147,18 +145,60 @@ void MirallConfigFile::writeOwncloudConfig( const QString& connection,
         pwd.clear();
     }
 
-    QByteArray pwdba = pwd.toUtf8();
-    settings.setValue( QLatin1String("passwd"), QVariant(pwdba.toBase64()) );
+#ifdef WITH_QTKEYCHAIN
+    // Password is stored to QtKeyChain now by default in CredentialStore
+    // The CredentialStore calls clearPasswordFromConfig after the creds
+    // were successfully wiritten to delete the passwd entry from config.
+    qDebug() << "Going to delete the password from settings file.";
+#else
+    if( !skipPwd )
+        writePassword( passwd );
+#endif
     settings.setValue( QLatin1String("nostoredpassword"), QVariant(skipPwd) );
     settings.sync();
     // check the perms, only read-write for the owner.
     QFile::setPermissions( file, QFile::ReadOwner|QFile::WriteOwner );
 
-    // inform the credential store about the password change.
-    CredentialStore::instance()->setCredentials( user, pwd );
+    // Store credentials temporar until the config is finalized.
+    CredentialStore::instance()->setCredentials( cloudsUrl, user, passwd );
 
 }
 
+// This method is called after the password was successfully stored into the
+// QKeyChain in CredentialStore.
+void MirallConfigFile::clearPasswordFromConfig( const QString& connection )
+{
+    const QString file = configFile();
+    QString con( defaultConnection() );
+    if( !connection.isEmpty() )
+        con = connection;
+
+    QSettings settings( file, QSettings::IniFormat);
+    settings.setIniCodec( "UTF-8" );
+    settings.beginGroup( con );
+    settings.remove(QLatin1String("passwd"));
+    settings.remove(QLatin1String("password"));
+    settings.sync();
+}
+
+bool MirallConfigFile::writePassword( const QString& passwd, const QString& connection )
+{
+    const QString file = configFile();
+    QString pwd( passwd );
+    QString con( defaultConnection() );
+    if( !connection.isEmpty() )
+        con = connection;
+
+    QSettings settings( file, QSettings::IniFormat);
+    settings.setIniCodec( "UTF-8" );
+
+    // store password into settings file.
+    settings.beginGroup( con );
+    QByteArray pwdba = pwd.toUtf8();
+    settings.setValue( QLatin1String("passwd"), QVariant(pwdba.toBase64()) );
+    settings.sync();
+}
+
 // set the url, called from redirect handling.
 void MirallConfigFile::setOwnCloudUrl( const QString& connection, const QString & url )
 {
@@ -267,70 +307,24 @@ int MirallConfigFile::remotePollInterval( const QString& connection ) const
   settings.beginGroup( con );
 
   int remoteInterval = settings.value( QLatin1String("remotePollInterval"), DEFAULT_REMOTE_POLL_INTERVAL ).toInt();
-  int localInterval  = settings.value(QLatin1String("localPollInterval"), DEFAULT_LOCAL_POLL_INTERVAL ).toInt();
-  if( remoteInterval < 2*localInterval ) {
-    qDebug() << "WARN: remote poll Interval should at least be twice as local poll interval!";
-  }
-  if( remoteInterval < 5000 || remoteInterval < localInterval ) {
-    qDebug() << "Remote Interval is smaller than local Interval";
+  if( remoteInterval < 5000) {
+    qDebug() << "Remote Interval is less than 5 seconds, reverting to" << DEFAULT_REMOTE_POLL_INTERVAL;
     remoteInterval = DEFAULT_REMOTE_POLL_INTERVAL;
   }
   return remoteInterval;
 }
 
-int MirallConfigFile::localPollInterval( const QString& connection ) const
+bool MirallConfigFile::passwordStorageAllowed( const QString& connection )
 {
-  QString con( connection );
-  if( connection.isEmpty() ) con = defaultConnection();
-
-  QSettings settings( configFile(), QSettings::IniFormat );
-  settings.setIniCodec( "UTF-8" );
-  settings.beginGroup( con );
-
-  int remoteInterval = settings.value( QLatin1String("remotePollInterval"), DEFAULT_REMOTE_POLL_INTERVAL ).toInt();
-  int localInterval  = settings.value(QLatin1String("localPollInterval"), DEFAULT_LOCAL_POLL_INTERVAL ).toInt();
-  if( remoteInterval < 2*localInterval ) {
-    qDebug() << "WARN: remote poll Interval should at least be twice as local poll interval!";
-  }
-  if( localInterval < 2500 || remoteInterval < localInterval ) {
-    qDebug() << "Remote Interval is smaller than local Interval";
-    localInterval = DEFAULT_LOCAL_POLL_INTERVAL;
-  }
-  return localInterval;
-}
-
-int MirallConfigFile::pollTimerExceedFactor( const QString& connection ) const
-{
-  QString con( connection );
-  if( connection.isEmpty() ) con = defaultConnection();
-
-  QSettings settings( configFile(), QSettings::IniFormat );
-  settings.setIniCodec( "UTF-8" );
-  settings.beginGroup( con );
-
-  int pte = settings.value( QLatin1String("pollTimerExeedFactor"), DEFAULT_POLL_TIMER_EXEED).toInt();
-
-  if( pte < 1 ) pte = DEFAULT_POLL_TIMER_EXEED;
-
-  return pte;
-}
-
-MirallConfigFile::CredentialType MirallConfigFile::credentialType() const
-{
-    QString con; /* ( connection ); */
-    /* if( connection.isEmpty() ) */ con = defaultConnection();
-    CredentialType ct = Settings;
+    QString con( connection );
+    if( connection.isEmpty() ) con = defaultConnection();
 
     QSettings settings( configFile(), QSettings::IniFormat );
     settings.setIniCodec( "UTF-8" );
     settings.beginGroup( con );
 
     bool skipPwd = settings.value( QLatin1String("nostoredpassword"), false ).toBool();
-    if( skipPwd ) {
-        ct = User;
-    }
-
-    return ct;
+    return !skipPwd;
 }
 
 QString MirallConfigFile::ownCloudPasswd( const QString& connection ) const
@@ -421,6 +415,7 @@ void MirallConfigFile::acceptCustomConfig()
     }
 
     QString srcConfig = configFile(); // this considers the custom handle
+
     _customHandle.clear();
     QString targetConfig = configFile();
     QString targetBak = targetConfig + QLatin1String(".bak");
@@ -441,69 +436,9 @@ void MirallConfigFile::acceptCustomConfig()
         }
     }
     QFile::remove( targetBak );
-}
 
-QVariant MirallConfigFile::customMedia( customMediaType type )
-{
-    QVariant re;
-    QString key;
-
-    if( type == oCSetupTop ) {
-        key = QLatin1String("oCSetupTop");
-    } else if( type == oCSetupSide ) {
-        key = QLatin1String("oCSetupSide");
-    } else if( type == oCSetupBottom) {
-        key = QLatin1String("oCSetupBottom");
-    } else if( type == oCSetupFixUrl ) {
-        key = QLatin1String("oCSetupFixUrl");
-    } else if( type == oCSetupResultTop ) {
-        key = QLatin1String("oCSetupResultTop");
-    } else {
-        qDebug() << "Wrong media type.";
-    }
-
-    if( !key.isEmpty() ) {
-        const QString customFile("custom.ini");
-        QFileInfo fi;
-
-#ifdef Q_OS_WIN32
-        fi.setFile( QApplication::applicationDirPath(), customFile );
-#endif
-#ifdef Q_OS_MAC
-        // exec path is inside the bundle
-        fi.setFile( QApplication::applicationDirPath(),
-                    QLatin1String("../Resources/") + customFile );
-#endif
-#ifdef Q_OS_LINUX
-        fi.setFile( QString("/etc/%1").arg(Theme::instance()->appName()), customFile );
-#endif
-        QSettings settings( fi.absoluteFilePath(), QSettings::IniFormat );
-
-        QString cfg = settings.fileName();
-        qDebug() << "Trying to read config ini file at " << cfg;
-
-        settings.setIniCodec( "UTF-8" );
-        settings.beginGroup(QLatin1String("GUICustomize"));
-        QString val = settings.value( key ).toString();
-
-        // if file is relative, prepend the application dir path.
-        QFileInfo checkFi(val);
-        if( !val.isEmpty() && checkFi.isRelative() ) {
-            checkFi.setFile( QApplication::applicationDirPath(), val );
-            val = checkFi.absoluteFilePath();
-        }
-
-        if( !val.isEmpty() ) {
-            QPixmap pix( val );
-            if( pix.isNull() ) {
-                // pixmap loading hasn't succeeded. We take the text instead.
-                re.setValue( val );
-            } else {
-                re.setValue( pix );
-            }
-        }
-    }
-    return re;
+    // inform the credential store about the password change.
+    CredentialStore::instance()->saveCredentials( );
 }
 
 void MirallConfigFile::setProxyType(int proxyType,
@@ -520,7 +455,7 @@ void MirallConfigFile::setProxyType(int proxyType,
     settings.setValue(QLatin1String("host"), host);
     settings.setValue(QLatin1String("port"), port);
     settings.setValue(QLatin1String("user"), user);
-    settings.setValue(QLatin1String("pass"), pass);
+    settings.setValue(QLatin1String("pass"), pass.toUtf8().toBase64());
 
     settings.sync();
 }

src/mirall/mirallconfigfile.h

@@ -30,19 +30,13 @@ class MirallConfigFile
 public:
     MirallConfigFile( const QString& appendix = QString() );
 
-    enum customMediaType {
-        oCSetupTop,      // ownCloud connect page
-        oCSetupSide,
-        oCSetupBottom,
-        oCSetupFixUrl,
-        oCSetupResultTop // ownCloud connect result page
-    };
-
-    enum CredentialType {
-        User = 0,
-        Settings,
-        KeyChain
-    };
+//    enum customMediaType {
+//        oCSetupTop,      // ownCloud connect page
+//        oCSetupSide,
+//        oCSetupBottom,
+//        oCSetupFixUrl,
+//        oCSetupResultTop // ownCloud connect result page
+//    };
 
     QString configPath() const;
     QString configFile() const;
@@ -69,22 +63,18 @@ public:
     QByteArray caCerts();
     void setCaCerts( const QByteArray& );
 
-    CredentialType credentialType() const;
+    bool passwordStorageAllowed(const QString &);
 
     QString ownCloudVersion() const;
     void setOwnCloudVersion( const QString& );
 
-    QVariant customMedia( customMediaType );
-
     // max count of lines in the log window
     int  maxLogLines() const;
 
     bool ownCloudSkipUpdateCheck( const QString& connection = QString() ) const;
 
-    /* Poll intervals in milliseconds */
-    int localPollInterval ( const QString& connection = QString() ) const;
+    /* Server poll interval in milliseconds */
     int remotePollInterval( const QString& connection = QString() ) const;
-    int pollTimerExceedFactor( const QString& connection = QString() ) const;
 
     // Custom Config: accept the custom config to become the main one.
     void acceptCustomConfig();
@@ -105,8 +95,11 @@ public:
     QString proxyPassword() const;
 
 protected:
+    // these classes can only be access from CredentialStore as a friend class.
     QString ownCloudPasswd( const QString& connection = QString() ) const;
     QString ownCloudUser( const QString& connection = QString() ) const;
+    void clearPasswordFromConfig( const QString& connect = QString() );
+    bool writePassword( const QString& passwd, const QString& connection = QString() );
 
 private:
     QVariant getValue(const QString& param, const QString& group) const;
@@ -116,6 +109,7 @@ private:
     static bool    _askedUser;
     static QString _oCVersion;
     QString        _customHandle;
+
 };
 
 }

src/mirall/miralltheme.cpp

@@ -62,7 +62,7 @@ QIcon mirallTheme::folderIcon( const QString& backend ) const
   return themeIcon( name );
 }
 
-QIcon mirallTheme::syncStateIcon( SyncResult::Status status ) const
+QIcon mirallTheme::syncStateIcon( SyncResult::Status status, bool sysTray ) const
 {
     QString statusIcon;
 
@@ -88,7 +88,7 @@ QIcon mirallTheme::syncStateIcon( SyncResult::Status status ) const
     default:
         statusIcon = QLatin1String("dialog-close");
     }
-    return themeIcon( statusIcon );
+    return themeIcon( statusIcon, sysTray );
 }
 
 

src/mirall/miralltheme.h

@@ -29,7 +29,7 @@ public:
     QPixmap splashScreen() const;
 
     QIcon   folderIcon( const QString& ) const;
-    QIcon   syncStateIcon(SyncResult::Status) const;
+    QIcon   syncStateIcon(SyncResult::Status, bool) const;
     QIcon   folderDisabledIcon() const;
     QIcon   applicationIcon() const;
 

src/mirall/owncloudfolder.cpp

@@ -17,6 +17,7 @@
 #include "mirall/mirallconfigfile.h"
 #include "mirall/owncloudinfo.h"
 #include "mirall/credentialstore.h"
+#include "mirall/logger.h"
 
 #include <csync.h>
 
@@ -35,6 +36,19 @@
 namespace Mirall {
 
 
+static QString replaceScheme(const QString &urlStr)
+{
+
+    QUrl url( urlStr );
+    if( url.scheme() == QLatin1String("http") ) {
+        url.setScheme( QLatin1String("owncloud") );
+    } else {
+        // connect SSL!
+        url.setScheme( QLatin1String("ownclouds") );
+    }
+    return url.toString();
+}
+
 ownCloudFolder::ownCloudFolder(const QString &alias,
                                const QString &path,
                                const QString &secondPath,
@@ -42,35 +56,16 @@ ownCloudFolder::ownCloudFolder(const QString &alias,
     : Folder(alias, path, secondPath, parent)
     , _secondPath(secondPath)
     , _thread(0)
-    , _localCheckOnly( false )
-    , _localFileChanges( false )
     , _csync(0)
-    , _pollTimerCnt(0)
     , _csyncError(false)
+    , _csyncUnavail(false)
     , _wipeDb(false)
-    , _lastSeenFiles(0)
 {
-#ifdef USE_INOTIFY
+    _notifier = new DownloadNotifier(QDir::fromNativeSeparators(path),
+                                     replaceScheme(secondPath), this);
+    connect(_notifier, SIGNAL(guiLog(QString,QString)), Logger::instance(), SIGNAL(guiLog(QString,QString)));
     qDebug() << "****** ownCloud folder using watcher *******";
     // The folder interval is set in the folder parent class.
-#else
-    /* If local polling is used, the polltimer of class Folder has to fire more
-     * often
-     * Set a local poll time of 2000 milliseconds, which results in a 30 seconds
-     * remote poll interval, defined in slotPollTimerRemoteCheck
-     */
-
-    MirallConfigFile cfgFile;
-
-    _pollTimer->stop();
-    connect( _pollTimer, SIGNAL(timeout()), this, SLOT(slotPollTimerRemoteCheck()));
-    setPollInterval( cfgFile.localPollInterval()- 500 + (int)( 1000.0*qrand()/(RAND_MAX+1.0) ) );
-    _pollTimerExceed = cfgFile.pollTimerExceedFactor();
-
-    _pollTimerCnt = _pollTimerExceed-1; // start the syncing quickly!
-    _pollTimer->start();
-    qDebug() << "****** ownCloud folder using local poll *******";
-#endif
 }
 
 ownCloudFolder::~ownCloudFolder()
@@ -78,13 +73,6 @@ ownCloudFolder::~ownCloudFolder()
 
 }
 
-/* Only used if INotify is not available. */
-void ownCloudFolder::slotPollTimerRemoteCheck()
-{
-    _pollTimerCnt++;
-    qDebug() << "**** Poll Timer for Folder " << alias() << " increase: " << _pollTimerCnt;
-}
-
 bool ownCloudFolder::isBusy() const
 {
     return ( _thread && _thread->isRunning() );
@@ -127,39 +115,21 @@ void ownCloudFolder::startSync(const QStringList &pathList)
     delete _thread;
     _errors.clear();
     _csyncError = false;
+    _csyncUnavail = false;
     _wipeDb = false;
 
     MirallConfigFile cfgFile;
 
-    QUrl url( _secondPath );
-    if( url.scheme() == QLatin1String("http") ) {
-        url.setScheme( QLatin1String("owncloud") );
-    } else {
-        // connect SSL!
-        url.setScheme( QLatin1String("ownclouds") );
-    }
-
-#ifdef USE_INOTIFY
-    // if there is a watcher and no polling, ever sync is remote.
-    _localCheckOnly = false;
     _syncResult.clearErrors();
-#else
-    _localCheckOnly = true;
-    if( _pollTimerCnt >= _pollTimerExceed || _localFileChanges ) {
-        _localCheckOnly = false;
-        _pollTimerCnt = 0;
-        _localFileChanges = false;
-        _syncResult.clearErrors();
-    }
-#endif
-    _syncResult.setLocalRunOnly( _localCheckOnly );
+    // we now have watchers for everything, so every sync is remote.
+    _syncResult.setLocalRunOnly( false );
     Folder::startSync( pathList );
 
+    QString url = replaceScheme(_secondPath);
 
-    qDebug() << "*** Start syncing url to ownCloud: " << url.toString() << ", with localOnly: " << _localCheckOnly;
-
+    qDebug() << "*** Start syncing url to ownCloud: " << url;
     _thread = new QThread(this);
-    _csync = new CSyncThread( path(), url.toString(), _localCheckOnly );
+    _csync = new CSyncThread( path(), url);
     _csync->moveToThread(_thread);
 
     QList<QNetworkProxy> proxies = QNetworkProxyFactory::proxyForQuery(QUrl(cfgFile.ownCloudUrl()));
@@ -170,15 +140,17 @@ void ownCloudFolder::startSync(const QStringList &pathList)
     _csync->setConnectionDetails( CredentialStore::instance()->user(),
                                   CredentialStore::instance()->password(),
                                   proxy );
+    qRegisterMetaType<SyncFileItemVector>("SyncFileItemVector");
+    connect( _csync, SIGNAL(treeWalkResult(const SyncFileItemVector&)),
+              this, SLOT(slotThreadTreeWalkResult(const SyncFileItemVector&)), Qt::QueuedConnection);
 
     connect(_csync, SIGNAL(started()),  SLOT(slotCSyncStarted()), Qt::QueuedConnection);
     connect(_csync, SIGNAL(finished()), SLOT(slotCSyncFinished()), Qt::QueuedConnection);
-    connect(_csync, SIGNAL(csyncError(const QString)), SLOT(slotCSyncError(const QString)), Qt::QueuedConnection);
+    connect(_csync, SIGNAL(csyncError(QString)), SLOT(slotCSyncError(QString)), Qt::QueuedConnection);
+    connect(_csync, SIGNAL(csyncUnavailable()), SLOT(slotCsyncUnavailable()), Qt::QueuedConnection);
+    connect(_csync, SIGNAL(fileReceived(QString)),
+            _notifier, SLOT(slotFileReceived(QString)), Qt::QueuedConnection);
 
-    qRegisterMetaType<SyncFileItemVector>("SyncFileItemVector");
-    qRegisterMetaType<WalkStats>("WalkStats");
-    connect( _csync, SIGNAL(treeWalkResult(SyncFileItemVector,WalkStats)),
-             this, SLOT(slotThreadTreeWalkResult(SyncFileItemVector, WalkStats)), Qt::QueuedConnection);
     _thread->start();
     QMetaObject::invokeMethod(_csync, "startSync", Qt::QueuedConnection);
 
@@ -190,45 +162,17 @@ void ownCloudFolder::slotCSyncStarted()
     emit syncStarted();
 }
 
-void ownCloudFolder::slotThreadTreeWalkResult(const SyncFileItemVector& items, const WalkStats& wStats )
-{
-    _items = items;
-    qDebug() << "Seen files: " << wStats.seenFiles;
-
-    /* check if there are happend changes in the file system */
-    qDebug() << "New     files: " << wStats.newFiles;
-    qDebug() << "Updated files: " << wStats.eval;
-    qDebug() << "Walked  files: " << wStats.seenFiles;
-    qDebug() << "Eval files: "    << wStats.eval;
-    qDebug() << "Removed files: " << wStats.removed;
-    qDebug() << "Renamed files: " << wStats.renamed;
-
-    if( ! _localCheckOnly ) _lastSeenFiles = 0;
-    _localFileChanges = false;
-
-#ifndef USE_INOTIFY
-    if( _lastSeenFiles > 0 && _lastSeenFiles != wStats.seenFiles ) {
-        qDebug() << "*** last seen files different from currently seen number " << _lastSeenFiles << "<>" << wStats.seenFiles << " => full Sync needed";
-        _localFileChanges = true;
-    }
-    if( (wStats.newFiles + wStats.eval + wStats.removed + wStats.renamed) > 0 ) {
-         qDebug() << "*** Local changes, lets do a full sync!" ;
-         _localFileChanges = true;
-    }
-    if( _pollTimerCnt < _pollTimerExceed ) {
-        qDebug() << "     *** No local changes, finalize, pollTimerCounter is "<< _pollTimerCnt ;
-    }
-#endif
-    _lastSeenFiles = wStats.seenFiles;
-
-}
-
 void ownCloudFolder::slotCSyncError(const QString& err)
 {
     _errors.append( err );
     _csyncError = true;
 }
 
+void ownCloudFolder::slotCsyncUnavailable()
+{
+    _csyncUnavail = true;
+}
+
 void ownCloudFolder::slotCSyncFinished()
 {
     qDebug() << "-> CSync Finished slot with error " << _csyncError;
@@ -240,17 +184,23 @@ void ownCloudFolder::slotCSyncFinished()
         _syncResult.setErrorStrings( _errors );
         qDebug() << "    * owncloud csync thread finished with error";
         if( _wipeDb ) wipe();
+    } else if (_csyncUnavail) {
+        _syncResult.setStatus(SyncResult::Unavailable);
     } else {
         _syncResult.setStatus(SyncResult::Success);
     }
 
-    if( ! _localCheckOnly ) _lastSeenFiles = 0;
     if( _thread && _thread->isRunning() ) {
         _thread->quit();
     }
     emit syncFinished( _syncResult );
 }
 
+void ownCloudFolder::slotThreadTreeWalkResult(const SyncFileItemVector& items)
+{
+    _syncResult.setSyncFileItemVector(items);
+}
+
 void ownCloudFolder::slotTerminateSync()
 {
     qDebug() << "folder " << alias() << " Terminating!";
@@ -330,54 +280,39 @@ void ownCloudFolder::wipe()
     _wipeDb = false;
 }
 
-SyncFileStatus ownCloudFolder::fileStatus( const QString& file )
+DownloadNotifier::DownloadNotifier(const QString &localPrefix, const QString &remotePrefix, QObject *parent)
+    : QObject(parent), _timer(new QTimer(this)), _items(0)
 {
-    if( file.isEmpty() ) return STATUS_NONE;
-    QFileInfo fi( path(), file );
+    _timer->setSingleShot(true);
+    connect(_timer, SIGNAL(timeout()), SLOT(sendResults()));
 
-    foreach( const SyncFileItem item, _items ) {
-        qDebug() << "FileStatus compare: " << item.file << " <> " << fi.absoluteFilePath();
+    _localPrefix = localPrefix;
+    _remotePrefix = remotePrefix;
+}
 
-        if( item.file == fi.absoluteFilePath() ) {
-            switch( item.instruction ) {
-            case   CSYNC_INSTRUCTION_NONE:
-                return STATUS_NONE;
-                break;
-            case   CSYNC_INSTRUCTION_EVAL:
-                return STATUS_EVAL;
-                break;
-            case   CSYNC_INSTRUCTION_RENAME:
-                return STATUS_RENAME;
-                break;
-            case   CSYNC_INSTRUCTION_NEW:
-                return STATUS_NEW;
-                break;
-            case   CSYNC_INSTRUCTION_CONFLICT:
-                return STATUS_CONFLICT;
-                break;
-            case   CSYNC_INSTRUCTION_IGNORE:
-                return STATUS_IGNORE;
-                break;
-            case   CSYNC_INSTRUCTION_SYNC:
-            case   CSYNC_INSTRUCTION_UPDATED:
-                return STATUS_SYNC;
-                break;
-            case   CSYNC_INSTRUCTION_STAT_ERROR:
-                return STATUS_STAT_ERROR;
-                break;
-            case   CSYNC_INSTRUCTION_ERROR:
-                return STATUS_ERROR;
-                break;
-            case   CSYNC_INSTRUCTION_DELETED:
-            case   CSYNC_INSTRUCTION_REMOVE:
-                return STATUS_REMOVE;
-                break;
-            default:
-                break;
-            }
-        }
-    }
-    return STATUS_NEW;
+void DownloadNotifier::slotFileReceived(const QString & url)
+{
+    if (_url.isEmpty())
+        _url = url;
+    _items++;
+    _timer->stop();
+    _timer->start(1000);
+}
+
+void DownloadNotifier::sendResults()
+{
+    QString file = _url;
+    file.replace(_remotePrefix, _localPrefix);
+    file = QDir::toNativeSeparators(QDir::cleanPath(file));
+    if (_items == 1)
+        emit guiLog(tr("New file available"), tr("'%1' has been synced to this machine.").arg(file));
+    else
+        emit guiLog(tr("New files available"), tr("'%1' and %n other file(s) have been synced to this machine.",
+                                                  "", _items-1).arg(file).arg(_items));
+
+    // reset
+    _items = 0;
+    _url = QString::null;
 }
 
 } // ns

src/mirall/owncloudfolder.h

@@ -23,6 +23,7 @@
 #include "mirall/csyncthread.h"
 
 class QProcess;
+class QTimer;
 
 namespace Mirall {
 
@@ -41,6 +42,25 @@ enum SyncFileStatus_s {
 };
 typedef SyncFileStatus_s SyncFileStatus;
 
+class DownloadNotifier : public QObject
+{
+    Q_OBJECT
+public:
+    DownloadNotifier(const QString &localPrefix, const QString &remotePrefix, QObject *parent = 0);
+public slots:
+    void slotFileReceived(const QString&);
+signals:
+    void guiLog(const QString&, const QString&);
+private slots:
+    void sendResults();
+private:
+    QTimer *_timer;
+    QString _url;
+    QString _localPrefix;
+    QString _remotePrefix;
+    int _items;
+};
+
 class ownCloudFolder : public Folder
 {
     Q_OBJECT
@@ -56,6 +76,7 @@ public:
 
     virtual void wipe();
 
+    /* get status about a singel file. */
     SyncFileStatus fileStatus( const QString& );
 
 public slots:
@@ -64,28 +85,24 @@ public slots:
 
 protected slots:
     void slotLocalPathChanged( const QString& );
+    void slotThreadTreeWalkResult(const SyncFileItemVector& );
 
 private slots:
     void slotCSyncStarted();
     void slotCSyncError(const QString& );
+    void slotCsyncUnavailable();
     void slotCSyncFinished();
-    void slotThreadTreeWalkResult(const SyncFileItemVector &, const WalkStats& );
-
-    void slotPollTimerRemoteCheck();
 
 private:
+    DownloadNotifier *_notifier;
     QString      _secondPath;
     QThread     *_thread;
     CSyncThread *_csync;
-    bool         _localCheckOnly;
-    bool         _localFileChanges;
-    int          _pollTimerCnt;
-    int          _pollTimerExceed;
     QStringList  _errors;
     bool         _csyncError;
+    bool         _csyncUnavail;
     bool         _wipeDb;
-    ulong        _lastSeenFiles;
-    QVector<SyncFileItem> _items;
+    SyncFileItemVector _items;
 };
 
 }

src/mirall/owncloudinfo.cpp

@@ -263,6 +263,11 @@ QString ownCloudInfo::configHandle(QNetworkReply *reply)
     return configHandle;
 }
 
+QList<QSslCertificate> ownCloudInfo::certificateChain() const
+{
+    return _certificateChain;
+}
+
 QUrl ownCloudInfo::redirectUrl(const QUrl& possibleRedirectUrl,
                                const QUrl& oldRedirectUrl) const {
     QUrl redirectUrl;
@@ -285,6 +290,10 @@ QUrl ownCloudInfo::redirectUrl(const QUrl& possibleRedirectUrl,
 void ownCloudInfo::slotReplyFinished()
 {
     QNetworkReply *reply = qobject_cast<QNetworkReply *>(sender());
+    QSslConfiguration sslConfig = reply->sslConfiguration();
+    if (!sslConfig.isNull()) {
+        _certificateChain = sslConfig.peerCertificateChain();
+    }
 
     if( ! reply ) {
         qDebug() << "ownCloudInfo: Reply empty!";

src/mirall/owncloudinfo.h

@@ -83,6 +83,12 @@ public:
      */
     QString configHandle(QNetworkReply *reply = 0);
 
+    /**
+     * Certificate chain of the connection est. with ownCloud.
+     * Empty if the connection is HTTP-based
+     */
+    QList<QSslCertificate> certificateChain() const;
+
 signals:
     // result signal with url- and version string.
     void ownCloudInfoFound( const QString&, const QString&, const QString&, const QString& );
@@ -129,6 +135,7 @@ private:
     QUrl                           _urlRedirectedTo;
     QHash<QNetworkReply*, QString> _directories;
     QHash<QNetworkReply*, QString> _configHandleMap;
+    QList<QSslCertificate>         _certificateChain;
     bool                           _certsUntrusted;
     int                            _authAttempts;
 };

src/mirall/owncloudsetupwizard.cpp

@@ -16,6 +16,7 @@
 #include "mirall/mirallconfigfile.h"
 #include "mirall/owncloudinfo.h"
 #include "mirall/folderman.h"
+#include "mirall/credentialstore.h"
 
 #include <QtCore>
 #include <QProcess>
@@ -88,6 +89,7 @@ void OwncloudSetupWizard::slotAssistantFinished( int result )
         qDebug() << "Rejected the new config, use the old!";
     } else if( result == QDialog::Accepted ) {
         qDebug() << "Config Changes were accepted!";
+        // save the user credentials and afterwards clear the cred store.
         cfg.acceptCustomConfig();
 
         // wipe all folder definitions so far.
@@ -391,15 +393,22 @@ void OwncloudSetupWizard::setupLocalSyncFolder()
 
     if( localFolderOk ) {
         _remoteFolder = Theme::instance()->defaultServerFolder();
-        if( !_remoteFolder.isEmpty() && createRemoteFolder( _remoteFolder ) ) {
-            // the creation started successfully, does not mean it will work
-            qDebug() << "Creation of remote folder started successfully.";
-        } else {
-            // the start of the http request failed.
-            _ocWizard->appendToResultWidget(tr("Creation of remote folder %1 could not be started.").arg(_remoteFolder));
-            qDebug() << "Creation of remote folder failed.";
-        }
+        slotCreateRemoteFolder(true);
+    }
+}
 
+void OwncloudSetupWizard::slotCreateRemoteFolder(bool credentialsOk )
+{
+    if( ! credentialsOk ) {
+        // User pressed cancel while being asked for password.
+        _ocWizard->appendToResultWidget("User canceled password dialog. Can not connect.");
+        return;
+    }
+
+    if( createRemoteFolder( _remoteFolder ) ) {
+        qDebug() << "Started remote folder creation ok";
+    } else {
+        _ocWizard->appendToResultWidget(tr("Creation of remote folder %1 could not be started.").arg(_remoteFolder));
     }
 }
 
@@ -450,14 +459,14 @@ void OwncloudSetupWizard::finalizeSetup( bool success )
         }
         _ocWizard->appendToResultWidget( QLatin1String(" "));
         _ocWizard->appendToResultWidget( QLatin1String("<p><font color=\"green\"><b>")
-                                                       + tr("Succesfully connected to %1!")
-                                                         .arg(Theme::instance()->appName())
-                                                       + QLatin1String("</b></font></p>"));
+                                         + tr("Succesfully connected to %1!")
+                                         .arg(Theme::instance()->appName())
+                                         + QLatin1String("</b></font></p>"));
         _ocWizard->appendToResultWidget( tr("Press Finish to permanently accept this connection."));
     } else {
         _ocWizard->appendToResultWidget(QLatin1String("<p><font color=\"red\">")
                                         + tr("Connection to %1 could not be established. Please check again.")
-                                          .arg(Theme::instance()->appName())
+                                        .arg(Theme::instance()->appName())
                                         + QLatin1String("</font></p>"));
     }
 }

src/mirall/owncloudsetupwizard.h

@@ -80,6 +80,8 @@ protected slots:
     void slotConnectToOCUrl( const QString& );
     void slotCreateOCLocalhost();
 
+    void slotCreateRemoteFolder(bool);
+
 private slots:
     void slotOwnCloudFound( const QString&, const QString&, const QString&, const QString& );
     void slotNoOwnCloudFound( QNetworkReply* );

src/mirall/owncloudtheme.cpp

@@ -18,8 +18,12 @@
 #include <QDebug>
 #include <QPixmap>
 #include <QIcon>
+#include <QStyle>
 #include <QApplication>
 
+#include "mirall/version.h"
+#include "config.h"
+
 namespace Mirall {
 
 ownCloudTheme::ownCloudTheme()
@@ -41,6 +45,39 @@ QString ownCloudTheme::configFileName() const
     return QLatin1String("owncloud.cfg");
 }
 
+QString ownCloudTheme::about() const
+{
+    QString devString;
+#ifdef GIT_SHA1
+    const QString githubPrefix(QLatin1String(
+                                   "    https://github.com/owncloud/mirall/commit/"));
+    const QString gitSha1(QLatin1String(GIT_SHA1));
+    devString = QCoreApplication::translate("ownCloudTheme::about()",
+                   "<p><small>Built from Git revision <a href=\"%1\">%2</a>"
+                   " on %3, %4<br>using OCsync %5 and Qt %6.</small><p>")
+            .arg(githubPrefix+gitSha1).arg(gitSha1.left(6))
+            .arg(__DATE__).arg(__TIME__)
+            .arg(MIRALL_STRINGIFY(LIBCSYNC_VERSION))
+            .arg(QT_VERSION_STR);
+#endif
+    return  QCoreApplication::translate("ownCloudTheme::about()",
+               "<p><b>%1 Client Version %2</b></p>"
+               "<p><b>Authors</b>"
+               "<br><a href=\"mailto:freitag@owncloud.com\">"
+               "Klaas Freitag</a>, ownCloud, Inc."
+               "<br><a href=\"mailto:danimo@owncloud.com\">"
+               "Daniel Molkentin</a>, ownCloud, Inc."
+               "<br><br>Based on Mirall by Duncan Mac-Vicar P.</p>"
+               "<p>For more information visit <a href=\"%3\">%4</a>.</p>"
+               "%7"
+               )
+            .arg(appName())
+            .arg(MIRALL_STRINGIFY(MIRALL_VERSION))
+            .arg("http://" MIRALL_STRINGIFY(APPLICATION_DOMAIN))
+            .arg(MIRALL_STRINGIFY(APPLICATION_DOMAIN))
+            .arg(devString);
+}
+
 QPixmap ownCloudTheme::splashScreen() const
 {
     return QPixmap(QLatin1String(":/mirall/resources/owncloud_splash.png"));
@@ -69,49 +106,45 @@ QIcon ownCloudTheme::folderIcon( const QString& backend ) const
 
 QIcon ownCloudTheme::trayFolderIcon( const QString& ) const
 {
-    return themeIcon( QLatin1String("owncloud-icon") );
+    QPixmap fallback = qApp->style()->standardPixmap(QStyle::SP_FileDialogNewFolder);
+    return QIcon::fromTheme("folder", fallback);
 }
 
-QIcon ownCloudTheme::syncStateIcon( SyncResult::Status status ) const
+QIcon ownCloudTheme::syncStateIcon( SyncResult::Status status, bool sysTray ) const
 {
     // FIXME: Mind the size!
     QString statusIcon;
 
     switch( status ) {
     case SyncResult::Undefined:
-        statusIcon = QLatin1String("owncloud-error");
-        break;
     case SyncResult::NotYetStarted:
-        statusIcon = QLatin1String("owncloud-icon");
+    case SyncResult::Unavailable:
+        statusIcon = QLatin1String("state-offline");
         break;
     case SyncResult::SyncRunning:
-        statusIcon = QLatin1String("owncloud-sync");
+        statusIcon = QLatin1String("state-sync");
         break;
     case SyncResult::Success:
-        statusIcon = QLatin1String("owncloud-sync-ok");
+        statusIcon = QLatin1String("state-ok");
         break;
     case SyncResult::Error:
-        statusIcon = QLatin1String("owncloud-error");
-        break;
     case SyncResult::SetupError:
-        statusIcon = QLatin1String("owncloud-error");
-        break;
     default:
-        statusIcon = QLatin1String("owncloud-error");
+        statusIcon = QLatin1String("state-error");
     }
 
-    return themeIcon( statusIcon );
+    return themeIcon( statusIcon, sysTray );
 }
 
 QIcon ownCloudTheme::folderDisabledIcon( ) const
 {
     // Fixme: Do we really want the dialog-canel from theme here?
-    return themeIcon( QLatin1String("owncloud-error") );
+    return themeIcon( QLatin1String("state-pause") );
 }
 
 QIcon ownCloudTheme::applicationIcon( ) const
 {
-    return themeIcon( QLatin1String("owncloud-icon") );
+    return themeIcon( QLatin1String("owncloud") );
 }
 
 }

src/mirall/owncloudtheme.h

@@ -26,11 +26,12 @@ public:
 
     virtual QString appName() const;
     QString configFileName() const;
+    QString about() const;
     QPixmap splashScreen() const;
 
     QIcon   folderIcon( const QString& ) const;
     QIcon   trayFolderIcon( const QString& ) const;
-    QIcon   syncStateIcon( SyncResult::Status ) const;
+    QIcon   syncStateIcon( SyncResult::Status, bool sysTray ) const;
     QIcon   folderDisabledIcon() const;
     QIcon   applicationIcon() const;
 

src/mirall/owncloudwizard.cpp

@@ -34,10 +34,10 @@ namespace Mirall
 
 void setupCustomMedia( QVariant variant, QLabel *label )
 {
-    if( ! label ) return;
+    if( !label ) return;
 
     QPixmap pix = variant.value<QPixmap>();
-    if( ! pix.isNull() ) {
+    if( !pix.isNull() ) {
         label->setPixmap(pix);
         label->setAlignment( Qt::AlignTop | Qt::AlignRight );
         label->setVisible(true);
@@ -65,10 +65,17 @@ OwncloudWelcomePage::OwncloudWelcomePage()
     content->setAlignment(Qt::AlignTop);
     content->setTextFormat(Qt::RichText);
     content->setWordWrap(true);
-    content->setText(tr("<p>In order to connect to your %1 server, you need to provide the server address "
-                        "as well as your credentials.</p><p>This wizard will guide you through the process.<p>"
-                        "<p>If you have not received this information, please contact your %1 hosting provider.</p>")
-                     .arg(Theme::instance()->appName()));
+    Theme *theme = Theme::instance();
+    if (theme->overrideServerUrl().isEmpty()) {
+        content->setText(tr("<p>In order to connect to your %1 server, you need to provide the server address "
+                            "as well as your credentials.</p><p>This wizard will guide you through the process.<p>"
+                            "<p>If you have not received this information, please contact your %1 provider.</p>")
+                         .arg(theme->appName()));
+    } else {
+        content->setText(tr("<p>In order to connect to your %1 server, you need to provide "
+                            "your credentials.</p><p>This wizard will guide you through "
+                            "the setup process.</p>").arg(theme->appName()));
+    }
 }
 
 
@@ -76,7 +83,7 @@ OwncloudSetupPage::OwncloudSetupPage()
 {
     _ui.setupUi(this);
 
-    setTitle(tr("Create the %1 Connection").arg(Theme::instance()->appName()));
+    setTitle(tr("Create Connection to %1").arg(Theme::instance()->appName()));
 
     connect(_ui.leUrl, SIGNAL(textChanged(QString)), SLOT(handleNewOcUrl(QString)));
 
@@ -90,6 +97,7 @@ OwncloudSetupPage::OwncloudSetupPage()
     _ui.cbSecureConnect->setEnabled(QSslSocket::supportsSsl());
 
     connect( _ui.lePassword, SIGNAL(textChanged(QString)), this, SIGNAL(completeChanged()));
+    connect( _ui.leUsername, SIGNAL(textChanged(QString)), this, SIGNAL(completeChanged()));
 
     connect( _ui.cbNoPasswordStore, SIGNAL(stateChanged(int)), this, SLOT(slotPwdStoreChanged(int)));
     connect( _ui.cbSecureConnect, SIGNAL(stateChanged(int)), this, SLOT(slotSecureConChanged(int)));
@@ -130,16 +138,15 @@ void OwncloudSetupPage::setupCustomization()
     _ui.topLabel->hide();
     _ui.bottomLabel->hide();
 
-    MirallConfigFile cfg;
-
-    QVariant variant = cfg.customMedia( MirallConfigFile::oCSetupTop );
+    Theme *theme = Theme::instance();
+    QVariant variant = theme->customMedia( Theme::oCSetupTop );
     setupCustomMedia( variant, _ui.topLabel );
-    variant = cfg.customMedia( MirallConfigFile::oCSetupSide );
+    variant = theme->customMedia( Theme::oCSetupSide );
     setupCustomMedia( variant, _ui.sideLabel );
-    variant = cfg.customMedia( MirallConfigFile::oCSetupBottom );
+    variant = theme->customMedia( Theme::oCSetupBottom );
     setupCustomMedia( variant, _ui.bottomLabel );
 
-    QString fixUrl = cfg.customMedia( MirallConfigFile::oCSetupFixUrl ).toString();
+    QString fixUrl = theme->overrideServerUrl();
     if( !fixUrl.isEmpty() ) {
         setOCUrl( fixUrl );
         _ui.leUrl->setEnabled( false );
@@ -458,9 +465,7 @@ void OwncloudWizardResultPage::setupCustomization()
     _ui.topLabel->setText( QString::null );
     _ui.topLabel->hide();
 
-    MirallConfigFile cfg;
-
-    QVariant variant = cfg.customMedia( MirallConfigFile::oCSetupResultTop );
+    QVariant variant = Theme::instance()->customMedia( Theme::oCSetupResultTop );
     setupCustomMedia( variant, _ui.topLabel );
 }
 

src/mirall/sslerrordialog.cpp

@@ -11,8 +11,9 @@
  * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
  * for more details.
  */
-#include "sslerrordialog.h"
 #include "mirall/mirallconfigfile.h"
+#include "mirall/utility.h"
+#include "mirall/sslerrordialog.h"
 
 #include <QtGui>
 #include <QtNetwork>
@@ -126,18 +127,6 @@ bool SslErrorDialog::setErrorList( QList<QSslError> errors )
     return false;
 }
 
-static QByteArray formatHash(const QByteArray &fmhash)
-{
-    QByteArray hash;
-    int steps = fmhash.length()/2;
-    for (int i = 0; i < steps; i++) {
-        hash.append(fmhash[i]);
-        hash.append(fmhash[i+1]);
-        hash.append(' ');
-    }
-    return hash;
-}
-
 QString SslErrorDialog::certDiv( QSslCertificate cert ) const
 {
     QString msg;
@@ -160,8 +149,10 @@ QString SslErrorDialog::certDiv( QSslCertificate cert ) const
 
     msg += QL("<p>");
 
-    QString md5sum = QString::fromAscii(formatHash(cert.digest(QCryptographicHash::Md5).toHex()));
-    QString sha1sum =  QString::fromAscii(formatHash(cert.digest(QCryptographicHash::Sha1).toHex()));
+    Utility util;
+
+    QString md5sum = util.formatFingerprint(cert.digest(QCryptographicHash::Md5).toHex());
+    QString sha1sum =  util.formatFingerprint(cert.digest(QCryptographicHash::Sha1).toHex());
     msg += tr("Fingerprint (MD5): <tt>%1</tt>").arg(md5sum) + QL("<br/>");
     msg += tr("Fingerprint (SHA1): <tt>%1</tt>").arg(sha1sum) + QL("<br/>");
     msg += QL("<br/>");

src/mirall/sslerrordialog.h

@@ -38,6 +38,7 @@ public:
     bool trustConnection();
 
     void setCustomConfigHandle( const QString& );
+
 signals:
     
 public slots:

src/mirall/statusdialog.cpp

@@ -18,6 +18,7 @@
 #include "mirall/owncloudinfo.h"
 #include "mirall/mirallconfigfile.h"
 #include "mirall/credentialstore.h"
+#include "mirall/fileitemdialog.h"
 
 #include <QtCore>
 #include <QtGui>
@@ -196,7 +197,12 @@ void FolderViewDelegate::paint(QPainter *painter, const QStyleOptionViewItem &op
       errorTextRect.setLeft( errorTextRect.left()+margin/2 +16);
       errorTextRect.setTop( errorTextRect.top()+margin/2 );
 
-      painter->drawText(errorTextRect, errorText);
+      int linebreak = errorText.indexOf(QLatin1String("<br"));
+      QString eText = errorText;
+      if(linebreak) {
+          eText = errorText.left(linebreak);
+      }
+      painter->drawText(errorTextRect, eText);
   }
 
   // painter->drawText(lastSyncRect, tr("Last Sync: %1").arg( statusText ));

src/mirall/statusdialog.h

@@ -61,7 +61,7 @@ class FolderViewDelegate : public QStyledItemDelegate
                       const QModelIndex& index );
 };
 
-class StatusDialog : public QDialog, public Ui::statusDialog
+class StatusDialog : public QDialog, protected Ui::statusDialog
 {
     Q_OBJECT
 public:

src/mirall/syncfileitem.h

@@ -0,0 +1,35 @@
+#ifndef SYNCFILEITEM_H
+#define SYNCFILEITEM_H
+
+#include <QVector>
+
+#include <csync.h>
+
+namespace Mirall {
+
+// FIXME: Unhack this.
+class SyncFileItem {
+public:
+    typedef enum {
+        None = 0,
+        Up,
+        Down } Direction;
+
+    SyncFileItem() {}
+
+    bool operator==(const SyncFileItem& item) const {
+        return item._file == this->_file;
+    }
+
+    // variables
+    QString _file;
+    QString _renameTarget;
+    csync_instructions_e _instruction;
+    Direction _dir;
+};
+
+typedef QVector<SyncFileItem> SyncFileItemVector;
+
+}
+
+#endif // SYNCFILEITEM_H

src/mirall/syncresult.cpp

@@ -58,6 +58,9 @@ QString SyncResult::statusString() const
         case SetupError:
             re = QLatin1String("SetupError");
             break;
+        case Unavailable:
+            re = QLatin1String("Not availabe");
+            break;
     }
     return re;
 }
@@ -65,6 +68,22 @@ QString SyncResult::statusString() const
 void SyncResult::setStatus( Status stat )
 {
     _status = stat;
+    _syncTime = QDateTime::currentDateTime();
+}
+
+void SyncResult::setSyncFileItemVector( const SyncFileItemVector& items )
+{
+    _syncItems = items;
+}
+
+SyncFileItemVector SyncResult::syncFileItemVector() const
+{
+    return _syncItems;
+}
+
+QDateTime SyncResult::syncTime() const
+{
+    return _syncTime;
 }
 
 void SyncResult::setErrorStrings( const QStringList& list )
@@ -93,16 +112,6 @@ void SyncResult::clearErrors()
     _errors.clear();
 }
 
-void SyncResult::setSyncChanges(const QHash< QString, QStringList >& changes)
-{
-    _syncChanges = changes;
-}
-
-QHash< QString, QStringList > SyncResult::syncChanges() const
-{
-    return _syncChanges;
-}
-
 bool SyncResult::localRunOnly() const
 {
     return _localRunOnly;

src/mirall/syncresult.h

@@ -17,6 +17,9 @@
 
 #include <QStringList>
 #include <QHash>
+#include <QDateTime>
+
+#include "mirall/syncfileitem.h"
 
 namespace Mirall
 {
@@ -31,7 +34,8 @@ public:
       SyncRunning,
       Success,
       Error,
-      SetupError
+      SetupError,
+      Unavailable
     };
 
     SyncResult();
@@ -43,25 +47,27 @@ public:
     QStringList errorStrings() const;
     void    clearErrors();
 
-    void    setSyncChanges( const QHash<QString, QStringList> &changes );
-    QHash<QString, QStringList> syncChanges() const;
+    // handle a list of changed items.
+    void    setSyncFileItemVector( const SyncFileItemVector& );
+    SyncFileItemVector syncFileItemVector() const;
 
     void setStatus( Status );
     Status status() const;
     QString statusString() const;
+    QDateTime syncTime() const;
 
     bool localRunOnly() const;
     void setLocalRunOnly( bool );
 private:
-    Status _status;
-    QHash<QString, QStringList> _syncChanges;
-
+    Status             _status;
+    SyncFileItemVector _syncItems;
+    QDateTime          _syncTime;
     /**
      * when the sync tool support this...
      */
-    QStringList _errors;
+    QStringList        _errors;
 
-    bool        _localRunOnly;
+    bool               _localRunOnly;
 };
 
 }

src/mirall/theme.cpp

@@ -14,6 +14,7 @@
 
 #include "theme.h"
 #include "version.h"
+#include "config.h"
 
 #include <QtCore>
 #include <QtGui>
@@ -21,6 +22,12 @@
 #include "mirall/miralltheme.h"
 #include "mirall/owncloudtheme.h"
 
+#ifdef THEME_INCLUDE
+#  define QUOTEME(M)       #M
+#  define INCLUDE_FILE(M)  QUOTEME(M)
+#  include INCLUDE_FILE(THEME_INCLUDE)
+#endif
+
 #include "config.h"
 
 namespace Mirall {
@@ -28,8 +35,10 @@ namespace Mirall {
 Theme* Theme::_instance = 0;
 
 Theme* Theme::instance() {
-    if (!_instance)
+    if (!_instance) {
         _instance = new THEME_CLASS;
+        _instance->_mono = false;
+    }
     return _instance;
 }
 
@@ -76,21 +85,40 @@ QIcon Theme::trayFolderIcon( const QString& backend ) const
  * helper to load a icon from either the icon theme the desktop provides or from
  * the apps Qt resources.
  */
-QIcon Theme::themeIcon( const QString& name ) const
+QIcon Theme::themeIcon( const QString& name, bool sysTray ) const
 {
+    QString flavor;
+    if (sysTray && _mono) {
+#ifdef Q_OS_MAC
+        flavor = QLatin1String("black");
+#else
+        flavor = QLatin1String("white");
+#endif
+    } else {
+        flavor = QLatin1String("colored");
+    }
+
     QIcon icon;
     if( QIcon::hasThemeIcon( name )) {
         // use from theme
         icon = QIcon::fromTheme( name );
     } else {
         QList<int> sizes;
-        sizes <<16 << 24 << 32 << 48 << 64 << 128;
+        sizes <<16 << 22 << 32 << 48 << 64 << 128;
         foreach (int size, sizes) {
-            QString pixmapName = QString::fromLatin1(":/mirall/resources/%1-%2.png").arg(name).arg(size);
+            QString pixmapName = QString::fromLatin1(":/mirall/theme/%1/%2-%3.png").arg(flavor).arg(name).arg(size);
             if (QFile::exists(pixmapName)) {
                 icon.addFile(pixmapName, QSize(size, size));
             }
         }
+        if (icon.isNull()) {
+            foreach (int size, sizes) {
+                QString pixmapName = QString::fromLatin1(":/mirall/resources/%1-%2.png").arg(name).arg(size);
+                if (QFile::exists(pixmapName)) {
+                    icon.addFile(pixmapName, QSize(size, size));
+                }
+            }
+        }
     }
     return icon;
 }
@@ -101,23 +129,69 @@ bool Theme::singleSyncFolder() const {
     return false;
 }
 
-/**
- * The default folder (without path) on the server at setup time.
- */
 QString Theme::defaultServerFolder() const
 {
     return QLatin1String("clientsync");
 }
 
-/**
- * The default folder (without path) on the client side at setup time.
- */
+QString Theme::overrideServerUrl() const
+{
+    return QString::null;
+}
+
 QString Theme::defaultClientFolder() const
 {
     return appName();
 }
 
+void Theme::setSystrayUseMonoIcons(bool mono)
+{
+    _mono = mono;
+}
 
+bool Theme::systrayUseMonoIcons() const
+{
+    return _mono;
+}
+
+QString Theme::about() const
+{
+    return QString::null;
+}
+
+QVariant Theme::customMedia( CustomMediaType type )
+{
+    QVariant re;
+    QString key;
+
+    switch ( type )
+    {
+    case oCSetupTop:
+        key = QLatin1String("oCSetupTop");
+        break;
+    case oCSetupSide:
+        key = QLatin1String("oCSetupSide");
+        break;
+    case oCSetupBottom:
+        key = QLatin1String("oCSetupBottom");
+        break;
+    case oCSetupResultTop:
+        key = QLatin1String("oCSetupResultTop");
+        break;
+    }
+
+    QString imgPath = QString::fromLatin1(":/mirall/theme/colored/%1.png").arg(key);
+    if ( QFile::exists( imgPath ) ) {
+        QPixmap pix( imgPath );
+        if( pix.isNull() ) {
+            // pixmap loading hasn't succeeded. We take the text instead.
+            re.setValue( key );
+        } else {
+            re.setValue( pix );
+        }
+    }
+    return re;
+}
 
 } // end namespace mirall
 

src/mirall/theme.h

@@ -30,6 +30,13 @@ class SyncResult;
 class Theme
 {
 public:
+    enum CustomMediaType {
+        oCSetupTop,      // ownCloud connect page
+        oCSetupSide,
+        oCSetupBottom,
+        oCSetupResultTop // ownCloud connect result page
+    };
+
     static Theme* instance();
 
     virtual QString appName() const = 0;
@@ -49,7 +56,7 @@ public:
     /**
       * get an sync state icon
       */
-    virtual QIcon   syncStateIcon( SyncResult::Status ) const = 0;
+    virtual QIcon   syncStateIcon( SyncResult::Status, bool sysTray = false ) const = 0;
 
     virtual QIcon   folderDisabledIcon() const = 0;
     virtual QPixmap splashScreen() const = 0;
@@ -64,6 +71,13 @@ public:
      */
     virtual bool singleSyncFolder() const;
 
+    /**
+     * Setting a value here will pre-define the server url.
+     *
+     * The respective UI controls will be disabled
+     */
+    virtual QString overrideServerUrl() const;
+
     /**
      * The default folder name without path on the server at setup time.
      */
@@ -80,8 +94,30 @@ public:
     virtual QString enforcedLocale() const { return QString::null; }
 
 
+    /**
+     * Override to use a string or a custom image name.
+     * The default implementation will try to look up
+     * :/mirall/theme/<type>.png
+     */
+    virtual QVariant customMedia( CustomMediaType type );
+
+    /**
+     * About dialog contents
+     */
+    virtual QString about() const;
+
+    /**
+     * Define if the systray icons should be using mono design
+     */
+    void setSystrayUseMonoIcons(bool mono);
+
+    /**
+     * Retrieve wether to use mono icons for systray
+     */
+    bool systrayUseMonoIcons() const;
+
 protected:
-    QIcon themeIcon(const QString& name) const;
+    QIcon themeIcon(const QString& name, bool sysTray = false) const;
     Theme() {}
 
 private:
@@ -89,7 +125,7 @@ private:
     Theme& operator=(Theme const&);
 
     static Theme* _instance;
-
+    bool _mono;
 
 };
 

src/mirall/utility.cpp

@@ -0,0 +1,33 @@
+/*
+ * Copyright (C) by Klaas Freitag <freitag@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#include "utility.h"
+namespace Mirall {
+
+QString Utility::formatFingerprint( const QByteArray& fmhash )
+{
+    QByteArray hash;
+    int steps = fmhash.length()/2;
+    for (int i = 0; i < steps; i++) {
+        hash.append(fmhash[i*2]);
+        hash.append(fmhash[i*2+1]);
+        hash.append(' ');
+    }
+
+    QString fp = QString::fromAscii( hash.trimmed() );
+    fp.replace(QChar(' '), QChar(':'));
+
+    return fp;
+}
+
+}

src/mirall/utility.h

@@ -0,0 +1,30 @@
+/*
+ * Copyright (C) by Klaas Freitag <freitag@owncloud.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * for more details.
+ */
+
+#ifndef UTILITY_H
+#define UTILITY_H
+
+#include <QString>
+#include <QByteArray>
+
+namespace Mirall {
+
+
+class Utility
+{
+public:
+    static QString formatFingerprint( const QByteArray& );
+};
+
+}
+#endif // UTILITY_H

sync-exclude.lst

@@ -29,3 +29,4 @@ Thumbs.db
 
 .htaccess
 
+Icon\r*

theme.qrc

@@ -0,0 +1,43 @@
+<RCC>
+    <qresource prefix="/mirall">
+        <file>theme/colored/state-sync-32.png</file>
+        <file>theme/colored/state-pause-32.png</file>
+        <file>theme/colored/state-ok-32.png</file>
+        <file>theme/colored/state-offline-32.png</file>
+        <file>theme/colored/state-error-32.png</file>
+        <file>theme/black/state-sync-32.png</file>
+        <file>theme/black/state-pause-32.png</file>
+        <file>theme/black/state-ok-32.png</file>
+        <file>theme/black/state-offline-32.png</file>
+        <file>theme/black/state-error-32.png</file>
+        <file>theme/white/state-sync-32.png</file>
+        <file>theme/white/state-pause-32.png</file>
+        <file>theme/white/state-ok-32.png</file>
+        <file>theme/white/state-offline-32.png</file>
+        <file>theme/white/state-error-32.png</file>
+
+        <file>theme/colored/state-sync-64.png</file>
+        <file>theme/colored/state-pause-64.png</file>
+        <file>theme/colored/state-ok-64.png</file>
+        <file>theme/colored/state-offline-64.png</file>
+        <file>theme/colored/state-error-64.png</file>
+        <file>theme/black/state-sync-64.png</file>
+        <file>theme/black/state-pause-64.png</file>
+        <file>theme/black/state-ok-64.png</file>
+        <file>theme/black/state-offline-64.png</file>
+        <file>theme/black/state-error-64.png</file>
+        <file>theme/white/state-sync-64.png</file>
+        <file>theme/white/state-pause-64.png</file>
+        <file>theme/white/state-ok-64.png</file>
+        <file>theme/white/state-offline-64.png</file>
+        <file>theme/white/state-error-64.png</file>
+
+        <file>theme/colored/owncloud-icon-22.png</file>
+        <file>theme/colored/owncloud-icon-32.png</file>
+        <file>theme/colored/owncloud-icon-48.png</file>
+        <file>theme/colored/owncloud-icon-64.png</file>
+        <file>theme/colored/owncloud-icon-128.png</file>
+
+        <file>theme/colored/owncloud-framed-64.png</file>
+    </qresource>
+</RCC>