UID validity diagnostics improvement
* Reduced the number of parameters passed to ui.validityproblem() because they were all just method-calls to the folder object, which is already passed as the first parameter (reduction of unnecessary complexity).
This commit is contained in:
parent
79a596be7b
commit
657b470d74
@ -1,6 +1,8 @@
|
|||||||
# Copyright (C) 2003 John Goerzen
|
# Copyright (C) 2003 John Goerzen
|
||||||
# <jgoerzen@complete.org>
|
# <jgoerzen@complete.org>
|
||||||
#
|
#
|
||||||
|
# Portions Copyright (C) 2007 David Favro <offlineimap@meta-dynamic.com>
|
||||||
|
#
|
||||||
# This program is free software; you can redistribute it and/or modify
|
# 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
|
# it under the terms of the GNU General Public License as published by
|
||||||
# the Free Software Foundation; either version 2 of the License, or
|
# the Free Software Foundation; either version 2 of the License, or
|
||||||
@ -189,13 +191,11 @@ def syncfolder(accountname, remoterepos, remotefolder, localrepos,
|
|||||||
# empty. So, in that case, just save it off.
|
# empty. So, in that case, just save it off.
|
||||||
if len(localfolder.getmessagelist()) or len(statusfolder.getmessagelist()):
|
if len(localfolder.getmessagelist()) or len(statusfolder.getmessagelist()):
|
||||||
if not localfolder.isuidvalidityok():
|
if not localfolder.isuidvalidityok():
|
||||||
ui.validityproblem(localfolder, localfolder.getsaveduidvalidity(),
|
ui.validityproblem(localfolder)
|
||||||
localfolder.getuidvalidity())
|
|
||||||
localrepos.restore_atime()
|
localrepos.restore_atime()
|
||||||
return
|
return
|
||||||
if not remotefolder.isuidvalidityok():
|
if not remotefolder.isuidvalidityok():
|
||||||
ui.validityproblem(remotefolder, remotefolder.getsaveduidvalidity(),
|
ui.validityproblem(remotefolder)
|
||||||
remotefolder.getuidvalidity())
|
|
||||||
localrepos.restore_atime()
|
localrepos.restore_atime()
|
||||||
return
|
return
|
||||||
else:
|
else:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user