Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug Report: Error! message at Browse EVE Fittings window #1223

Closed
pumbah opened this issue Jun 8, 2017 · 3 comments
Closed

Bug Report: Error! message at Browse EVE Fittings window #1223

pumbah opened this issue Jun 8, 2017 · 3 comments
Labels
bug Confirmed to be a bug fixed This issue has been fixed! Oh joy!

Comments

@pumbah
Copy link

pumbah commented Jun 8, 2017

Bug Report

Expected behavior:

Cascade submenu

Actual behavior:

ERROR
OS version: Windows-7-6.1.7601-SP1
Python: 2.7.10
wxPython: 3.0.2.0
SQLAlchemy: 1.0.5
Logbook: 1.0.0
pyfa version: 1.29.3 Stable - YC119.5 1.0
pyfa root: C:\Program Files (x86)\pyfa
save path: C:\Users\username.pyfa
fs encoding: mbcs

EXCEPTION: expected string or buffer

File "C:\Program Files (x86)\pyfa\library.zip\gui\crestFittings.py", line 410, in displayFit
fit = json.loads(self.fittingsTreeCtrl.GetPyData(selection))
File "C:\python-2.7.10\lib\json_init_.py", line 338, in loads
File "C:\python-2.7.10\lib\json\decoder.py", line 366, in decode

Detailed steps to reproduce:

  1. logged on to Crest
  2. And then menu click Crest/Browse Fittings.
  3. A new window Browse EVE Fittings appear
  4. Then Fetch Fits
  5. Doubleclick any entries in the list of the fittings shipname text besides the cascading + generates the above error message

Fits involved in EFT format (Edit > To Clipboard > EFT):

Release or development git branch? Please note the release version or commit hash:

PYFA Version 1.29.3.

Operating system and version (eg: Windows 10, OS X 10.9, OS X 10.11, Ubuntu 16.10):

OS version: Windows-7-6.1.7601-SP1

Other relevant information:

@pumbah pumbah changed the title ERROR! Bug Report: Error! message at Browse EVE Fittings window Jun 8, 2017
@blitzmann
Copy link
Collaborator

Ah, sweet, thanks for the bug report. Thankfully this is an easy fix. I plan to have a point release out tonight to address some issues with old deleted fits hanging around, hope to include this as well. :)

@blitzmann blitzmann added bug Confirmed to be a bug fixed This issue has been fixed! Oh joy! labels Jun 8, 2017
@blitzmann
Copy link
Collaborator

Fixed with d319b74

@pumbah
Copy link
Author

pumbah commented Jun 9, 2017

yw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed to be a bug fixed This issue has been fixed! Oh joy!
Projects
None yet
Development

No branches or pull requests

2 participants