Code review comment for lp:~tomasgroth/openlp/mupdf

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Also, if I start up without a configuration, I get the following exception (which I don't get in trunk):

Traceback (most recent call last):
  File "openlp.py", line 45, in <module>
    main()
  File "/home/raoul/Projects/OpenLP/mupdf/openlp/core/__init__.py", line 329, in main
    sys.exit(application.run(qt_args))
  File "/home/raoul/Projects/OpenLP/mupdf/openlp/core/__init__.py", line 150, in run
    self.main_window.first_time()
  File "/home/raoul/Projects/OpenLP/mupdf/openlp/core/ui/mainwindow.py", line 636, in first_time
    plugin.first_time()
  File "/home/raoul/Projects/OpenLP/mupdf/openlp/plugins/songs/songsplugin.py", line 264, in first_time
    self.on_tools_reindex_item_triggered()
  File "/home/raoul/Projects/OpenLP/mupdf/openlp/plugins/songs/songsplugin.py", line 180, in on_tools_reindex_item_triggered
    self.media_item.on_search_text_button_clicked()
AttributeError: 'NoneType' object has no attribute 'on_search_text_button_clicked'

review: Needs Fixing

« Back to merge proposal