OpenLP

Can't delete/add a projector...

Bug #1625767 reported by Andrew Reyes
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenLP
Fix Committed
Medium
Unassigned
OpenLP 2.9.1

Bug Description

When trying to add a projector through the projector manager it won't connect and when opening OpenLP with the 'connect to projector on startup' option enabled it gives an error window. I also cannot delete projector as the same window pops up with an error. So all in all it won't connect.

PC Specs:
-Arch Linux (kernel 4.7.4)
-OpenLP-bzr2496

Steps to reproduce issue:
1. Click on add projector
2. Fill in info to projector
3. Click on connect...nothing happens
4. Click on Settins/Configure OpenLP/Projectors and enable
   the 'Connect to projectors on startup' option
5. Close OpenLP
6. Open OpenLP and error window pops up.
7. Delete the projector in Projector Manager the same popup window shows up.

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

Current revision is 2695. Please update.

Changed in openlp:
status: New → Incomplete
Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Hi Andrew,

This is a bit weird, your bug report says revision 2496 but the log file says 2695. Can you run "bzr revno" on your directory and tell me what revision it is?

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

Er, 2696, I mean. Your bug report version dictionary is very weird. How are you running OpenLP?

Revision history for this message
Ken Roberts (alisonken1) wrote :

Delete the following lines from the listed files:

lib/projector/pjlink1.py: projectorNetwork = QtCore.pyqtSignal(int) # Projector network activity
lib/projector/pjlink1.py: self.projectorNetwork.emit(S_NETWORK_RECEIVED)
lib/projector/pjlink1.py: self.projectorNetwork.emit(S_NETWORK_SENDING)
lib/projector/pjlink1.py: self.projectorNetwork.emit(S_NETWORK_SENDING)
ui/projector/manager.py: item.link.projectorNetwork.connect(self.update_status)

ui/projector/manager.py:
    try:
        projector.link.projectorNetwork.disconnect(self.update_status)
    except (AttributeError, TypeError):
        pass

These lines were originally put in for network activity for each projector; however, the network activity notification will be changed to a generic network notification for all network activity and these lines are now just noise.

Raoul Snyman (raoul-snyman)
Changed in openlp:
status: Incomplete → Confirmed
Revision history for this message
Ken Roberts (alisonken1) wrote :

The above change (delete all references to projectorNetwork()) can be removed from both 2.4 and trunk.

Revision history for this message
Andrew Reyes (dsreyes1014) wrote :

Raoul Snyman: Sorry about the typo. That's my fault. Also I'm installing OpenLP on arch. It's the bzr version of OpenLP not the 2.4.2 release.

Ken Roberts: I will remove and see how this works. Thanks guys and I will report back.

Revision history for this message
Andrew Reyes (dsreyes1014) wrote :

Ken Roberts: It still won't connect at all to the projector after deleting code in said files.

Revision history for this message
Andrew Reyes (dsreyes1014) wrote :

Edit: Actually it was a network reconfig I had to do on my end. The hack for the files works and projector is working as expected except for the blackout button. That does nothing when you click on it. Aside from that it's pretty much fully functional. Thanks Ken. Looking forward to 2.6.

Revision history for this message
Andrew Reyes (dsreyes1014) wrote :

Update: Did an update to our Arch system which included python-setuptools version 1:27.2.0-1 -> 1:27.3.0-1, sip version 4,18-1 -> 4.18.1-1, and python-sip version 4.18-1 -> 4.18.1-1 and projector stopped working. It will no longer connect.

Revision history for this message
Andrew Reyes (dsreyes1014) wrote :

Edit: Sorry guys. I seem to be having networking issues with our router. The hack still works.

Revision history for this message
Phill (phill-ridout) wrote :

Just walked in to this on trunk 2700 Ubuntu. Unable to produce it reliably.

Looks like the lines mentioned by Ken still need removing!

Revision history for this message
Ken Roberts (alisonken1) wrote :

Phil:

When you can, try setting debug and attach the log so I can have a look at it.

Ken Roberts (alisonken1)
Changed in openlp:
importance: Undecided → Medium
status: Confirmed → Fix Committed
Tim Bentley (trb143)
Changed in openlp:
milestone: none → 2.9.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.