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

Crash on loading CSV layouts #63

Closed
hsoft opened this issue Jun 22, 2013 · 4 comments
Closed

Crash on loading CSV layouts #63

hsoft opened this issue Jun 22, 2013 · 4 comments
Labels
bug

Comments

@hsoft
Copy link
Owner

@hsoft hsoft commented Jun 22, 2013

@@@
Application Identifier: com.hardcoded_software.moneyguru
Application Version: 1.6.1

Traceback (most recent call last):
File "/Users/dietrich/Desktop/diverses/moneyGuru.app/Contents/Resources/mg_cocoa.plugin/Contents/Resources/mg_cocoa.py", line 302, in initWithCocoa_pyParent_
File "moneyguru/gui/csv_options.pyc", line 103, in init
File "moneyguru/gui/csv_options.pyc", line 89, in preference2layout
TypeError: string indices must be integers
@@@

I can't reproduce it, and when I asked the user what was in his defaults, there was nothing:

defaults read com.hardcoded_software.moneyguru CSVLayouts
2009-09-02 22:43:27.680 defaults[11095:10b]
The domain/default pair of (com.hardcoded_software.moneyguru, CSVLayouts) does not exist

@hsoft hsoft closed this Jun 22, 2013
@hsoft
Copy link
Owner Author

@hsoft hsoft commented Jun 22, 2013

I got something different from someone who had the same error:

@@@
defaults read com.hardcoded_software.moneyguru CSVLayouts
(lp0
(dp1
S'excluded_lines'
p2
(lp3
sS'name'
p4
S'Default'
p5
sS'columns'
p6
(lp7
sa.
@@@

Seems like old-style pref (pickle) is not being handled correctly in 1.6.

@hsoft
Copy link
Owner Author

@hsoft hsoft commented Jun 22, 2013

[milestone:id#49656 bulk edit command]

@hsoft
Copy link
Owner Author

@hsoft hsoft commented Jun 22, 2013

(from [113]) [#63 state:fixed] Fixed a bug causing a crash on launch if the csv layout prefs were invalid.

@hsoft
Copy link
Owner Author

@hsoft hsoft commented Jun 22, 2013

[tagged:"csv" bulk edit command]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant