TheTash / l5rcm

Automatically exported from code.google.com/p/l5rcm
GNU General Public License v3.0
0 stars 0 forks source link

3.8.1-3 does not allow editing advantage/disadvantage text fields in some cases #183

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create a new character
2. Go to advantage/disadvantage tab and add advantage Material/Gaijin Gear
3. Select and edit it. Text can be edited and saved with "Ok".
4. Add disadvantage Material/Ruined City Survivor.
5. Select and edit it. Text can be edited but not saved. After entering
text and hitting "Ok" in "Add Disadvantage" dialogue, console reports:

Traceback (most recent call last):
  File "/opt/l5rcm/dialogs/perkseldlg.py", line 158, in on_perk_select
    self.perk_id = perk.id
AttributeError: 'unicode' object has no attribute 'id'
Traceback (most recent call last):
  File "/opt/l5rcm/dialogs/perkseldlg.py", line 203, in on_accept
    self.update_perk()
  File "/opt/l5rcm/dialogs/perkseldlg.py", line 199, in update_perk
    self.item.extra = self.tx_notes.toPlainText()
AttributeError: 'NoneType' object has no attribute 'extra'

What is the expected output? What do you see instead?

Expected to save edited text. No text is saved.

What version of the product are you using? On what operating system?

3.8.1-3 (although the internal product info tab says "3.8.0") on
Debian 7 amd64.

Original issue reported on code.google.com by jplgp...@gmail.com on 30 May 2013 at 8:15

GoogleCodeExporter commented 9 years ago

Original comment by oppifjel...@gmail.com on 30 May 2013 at 9:52

GoogleCodeExporter commented 9 years ago
This is solved with version 3.8.2 on all platforms

Original comment by oppifjel...@gmail.com on 7 Jun 2013 at 6:50

GoogleCodeExporter commented 9 years ago

Original comment by oppifjel...@gmail.com on 10 Jun 2013 at 8:29

GoogleCodeExporter commented 9 years ago
Confirmed, no longer seeing this problem. Thank you!

Original comment by jplgp...@gmail.com on 10 Jun 2013 at 8:45