first time wizard crashes

Bug #1742910 reported by Simon Hanna on 2018-01-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenLP
Critical
Raoul Snyman

Bug Description

Traceback (most recent call last):
  File "/home/sim/dev/openlp/better-threading/openlp/core/ui/firsttimeform.py", line 307, in on_current_id_changed
    self._download_index()
  File "/home/sim/dev/openlp/better-threading/openlp/core/ui/firsttimeform.py", line 260, in _download_index
    run_thread(worker, thread_name)
  File "/home/sim/dev/openlp/better-threading/openlp/core/threading.py", line 54, in run_thread
    if thread_name in main_window.threads:
AttributeError: 'NoneType' object has no attribute 'threads'

Using the better-threading branch, with no data or configs for OpenLP present in my home running on Archlinux.

Related branches

Ken Roberts (alisonken1) wrote :

Confirmed on Fedora 25 as well.

tags: added: threads
tags: added: ftw
Changed in openlp:
importance: Undecided → Critical
Simon Hanna (thelinuxguy) wrote :

The issue seems to be that the threads are all stored in the mainwindow which isn't always there.

If you run the first time wizard in the very beginning, no mainwindow exists.

So the threads should probably be attached to something that is created much earlier, before anything else is done, so it's guaranteed to be always there.

Changed in openlp:
assignee: nobody → Raoul Snyman (raoul-snyman)
Tim Bentley (trb143) on 2018-01-13
Changed in openlp:
milestone: none → 2.9.1
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers