OpenLP

Code review comment for lp:~phill-ridout/openlp/overwrite

  1. overwrite
  2. Merge into trunk
Revision history for this message
Jonathan Corwin (j-corwin) wrote :

Gave it permission to overwrite a theme and got:

Traceback (most recent call last):
  File "C:\Users\Jonathan\Documents\projects\openlp\overwrite\openlp\core\ui\thememanager.py", line 431, in onImportTheme
    self.unzipTheme(file, self.path)
  File "C:\Users\Jonathan\Documents\projects\openlp\overwrite\openlp\core\ui\thememanager.py", line 599, in unzipTheme
    self.generateAndSaveImage(dir, theme_name, theme)
  File "C:\Users\Jonathan\Documents\projects\openlp\overwrite\openlp\core\ui\thememanager.py", line 708, in generateAndSaveImage
    frame = self.generateImage(theme)
  File "C:\Users\Jonathan\Documents\projects\openlp\overwrite\openlp\core\ui\thememanager.py", line 740, in generateImage
    return self.mainWindow.renderer.generate_preview(
AttributeError: 'ThemeManager' object has no attribute 'mainWindow'

review: Needs Fixing

« Back to merge proposal