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

Legacy Webapp: traceback when rendering survivor __repr__ #400

Closed
toconnell opened this Issue Nov 8, 2017 · 4 comments

Comments

Projects
None yet
1 participant
@toconnell
Owner

toconnell commented Nov 8, 2017

From auto-email:

[57aa6ab9421aa94a7cfb040d] was logged out of the webapp instance on ubuntu-trusty-1 due to a render failure at 2017-11-08 19:00:16.204841.

 The traceback from the exception was this:
Traceback (most recent call last):
File "/home/toconnell/kdm-manager/v1/session.py", line 64, in wrapper
 return func(self, *args, **kwargs)
File "/home/toconnell/kdm-manager/v1/session.py", line 429, in process_params
 S.remove()
File "/home/toconnell/kdm-manager/v1/assets.py", line 1019, in remove
 S.remove()
File "/home/toconnell/kdm-manager/v1/assets.py", line 410, in remove
 self.logger.info("[%s] removing survivor %s" % (self.User, self))
File "/home/toconnell/kdm-manager/v1/assets.py", line 265, in __repr__
 return self.get_name_and_id(include_id=False, include_sex=True)
File "/home/toconnell/kdm-manager/v1/assets.py", line 430, in get_name_and_id
 return " ".join(output)
TypeError: sequence item 0: expected string, list found

No idea why this is happening (or why now).

@toconnell toconnell added bug v2 labels Nov 8, 2017

@toconnell toconnell self-assigned this Nov 8, 2017

@toconnell

This comment has been minimized.

Show comment
Hide comment
@toconnell

toconnell Nov 8, 2017

Owner

I'm unable to replicate this by removing a random survivor in a test settlement. Gonna have to clone the user down.

Owner

toconnell commented Nov 8, 2017

I'm unable to replicate this by removing a random survivor in a test settlement. Gonna have to clone the user down.

@toconnell

This comment has been minimized.

Show comment
Hide comment
@toconnell

toconnell Nov 8, 2017

Owner

Found an unrelated bug where removed survivors were being retrieved by the API after removal.

Owner

toconnell commented Nov 8, 2017

Found an unrelated bug where removed survivors were being retrieved by the API after removal.

@toconnell

This comment has been minimized.

Show comment
Hide comment
@toconnell

toconnell Nov 8, 2017

Owner

I just removed every survivor in this guy's settlment and was not able to reproduce. I really hope he emails me.

Owner

toconnell commented Nov 8, 2017

I just removed every survivor in this guy's settlment and was not able to reproduce. I really hope he emails me.

@toconnell toconnell added the wontfix label Nov 13, 2017

@toconnell

This comment has been minimized.

Show comment
Hide comment
@toconnell

toconnell Nov 13, 2017

Owner

It's been almost a week and no recurrence or user contact. Closing this one.

Owner

toconnell commented Nov 13, 2017

It's been almost a week and no recurrence or user contact. Closing this one.

@toconnell toconnell closed this Nov 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment