forked from openlp/openlp
dialog show only once (when more files are missing)
This commit is contained in:
commit
0d2e8ff12d
@ -41,7 +41,7 @@ class ImagePlugin(Plugin):
|
|||||||
self.icon = build_icon(self.icon_path)
|
self.icon = build_icon(self.icon_path)
|
||||||
|
|
||||||
def getMediaManagerItem(self):
|
def getMediaManagerItem(self):
|
||||||
# Create the MediaManagerItem object
|
# Create the MediaManagerItem object.
|
||||||
return ImageMediaItem(self, self, self.icon)
|
return ImageMediaItem(self, self, self.icon)
|
||||||
|
|
||||||
def about(self):
|
def about(self):
|
||||||
|
@ -166,29 +166,39 @@ class ImageMediaItem(MediaManagerItem):
|
|||||||
service_item.add_capability(ItemCapabilities.AllowsAdditions)
|
service_item.add_capability(ItemCapabilities.AllowsAdditions)
|
||||||
# force a nonexistent theme
|
# force a nonexistent theme
|
||||||
service_item.theme = -1
|
service_item.theme = -1
|
||||||
|
missing_items = []
|
||||||
|
missing_items_filenames = []
|
||||||
for item in items:
|
for item in items:
|
||||||
bitem = self.listView.item(item.row())
|
bitem = self.listView.item(item.row())
|
||||||
filename = unicode(bitem.data(QtCore.Qt.UserRole).toString())
|
filename = unicode(bitem.data(QtCore.Qt.UserRole).toString())
|
||||||
if os.path.exists(filename):
|
if not os.path.exists(filename):
|
||||||
(path, name) = os.path.split(filename)
|
missing_items.append(item)
|
||||||
service_item.add_from_image(filename, name)
|
missing_items_filenames.append(filename)
|
||||||
# We have only one image, which is no longer present.
|
for item in missing_items:
|
||||||
elif len(items) == 1:
|
items.remove(item)
|
||||||
QtGui.QMessageBox.critical(
|
# We cannot continue, as all images do not exist.
|
||||||
self, translate('ImagePlugin.MediaItem',
|
if not items:
|
||||||
'Missing Image'),
|
QtGui.QMessageBox.critical(self,
|
||||||
|
translate('ImagePlugin.MediaItem', 'Missing Image(s)'),
|
||||||
unicode(translate('ImagePlugin.MediaItem',
|
unicode(translate('ImagePlugin.MediaItem',
|
||||||
'The image %s no longer exists.')) % filename)
|
'The following image(s) no longer exist: %s')) %
|
||||||
|
u'\n'.join(missing_items_filenames))
|
||||||
return False
|
return False
|
||||||
# We have more than one item, but a file is missing.
|
# We have missing as well as existing images. We ask what to do.
|
||||||
elif QtGui.QMessageBox.question(self,
|
elif missing_items and QtGui.QMessageBox.question(self,
|
||||||
translate('ImagePlugin.MediaItem', 'Missing Image'),
|
translate('ImagePlugin.MediaItem', 'Missing Image(s)'),
|
||||||
unicode(translate('ImagePlugin.MediaItem', 'The image %s '
|
unicode(translate('ImagePlugin.MediaItem', 'The following '
|
||||||
'no longer exists. Do you want to add the other images '
|
'image(s) no longer exist: %s\nDo you want to add the other '
|
||||||
'anyway?')) % filename,
|
'images anyway?')) % u'\n'.join(missing_items_filenames),
|
||||||
QtGui.QMessageBox.StandardButtons(QtGui.QMessageBox.No |
|
QtGui.QMessageBox.StandardButtons(QtGui.QMessageBox.No |
|
||||||
QtGui.QMessageBox.Yes)) == QtGui.QMessageBox.No:
|
QtGui.QMessageBox.Yes)) == QtGui.QMessageBox.No:
|
||||||
return False
|
return False
|
||||||
|
# Continue with the existing images.
|
||||||
|
for item in items:
|
||||||
|
bitem = self.listView.item(item.row())
|
||||||
|
filename = unicode(bitem.data(QtCore.Qt.UserRole).toString())
|
||||||
|
(path, name) = os.path.split(filename)
|
||||||
|
service_item.add_from_image(filename, name)
|
||||||
return True
|
return True
|
||||||
else:
|
else:
|
||||||
return False
|
return False
|
||||||
@ -198,6 +208,7 @@ class ImageMediaItem(MediaManagerItem):
|
|||||||
self.parent.liveController.display.resetImage()
|
self.parent.liveController.display.resetImage()
|
||||||
|
|
||||||
def onReplaceClick(self):
|
def onReplaceClick(self):
|
||||||
|
# TODO: Check if image exists.
|
||||||
if check_item_selected(self.listView,
|
if check_item_selected(self.listView,
|
||||||
translate('ImagePlugin.MediaItem',
|
translate('ImagePlugin.MediaItem',
|
||||||
'You must select an image to replace the background with.')):
|
'You must select an image to replace the background with.')):
|
||||||
|
Loading…
Reference in New Issue
Block a user