Opened 6 years ago

Closed 6 years ago

#7960 closed defect (fixed)

AttributeError: 'module' object has no attribute 'camel_case'

Reported by: gsathya Owned by: atagar
Priority: Medium Milestone:
Component: Core Tor/Nyx Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

I have no idea how I triggered this. I was randomly pressing keys trying to edit the torrc in arm and it crashed.

gsathya@nub:~/arm$ ./arm
Traceback (most recent call last):
  File "./src/starter.py", line 478, in <module>
    cli.controller.startTorMonitor(time.time() - initTime)
  File "/home/gsathya/arm/src/cli/controller.py", line 564, in startTorMonitor
    curses.wrapper(drawTorMonitor, startTime)
  File "/usr/lib/python2.6/curses/wrapper.py", line 43, in wrapper
    return func(stdscr, *args, **kwds)
  File "/home/gsathya/arm/src/cli/controller.py", line 635, in drawTorMonitor
    cli.menu.menu.showMenu()
  File "/home/gsathya/arm/src/cli/menu/menu.py", line 80, in showMenu
    menu = cli.menu.actions.makeMenu()
  File "/home/gsathya/arm/src/cli/menu/actions.py", line 23, in makeMenu
    baseMenu.add(makeViewMenu())
  File "/home/gsathya/arm/src/cli/menu/actions.py", line 89, in makeViewMenu
    label = " / ".join([str_tools.camel_case(panel.getName()) for panel in pagePanels])
AttributeError: 'module' object has no attribute 'camel_case'

(I can't think of a proper title for this ticket, please feel free to change it)

Child Tickets

Attachments (2)

0001-Fix-AttributeError-in-makeViewMenu.patch (1006 bytes) - added by gsathya 6 years ago.
0001-Change-all-camel_case-to-to_camel_case.patch (1.6 KB) - added by gsathya 6 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by gsathya

Running on latest master branch (4232a73f628836e210af05470c0466a9d8db6ad6)

Changed 6 years ago by gsathya

comment:2 Changed 6 years ago by gsathya

Status: newneeds_review

Attached patch. I think this fixes it.

comment:3 Changed 6 years ago by gsathya

Found more occurrences of camel_case(). Attached patch changing it to to_camel_case()

comment:4 Changed 6 years ago by gsathya

Ok I figured out how to reproduce this. Pressing 'm' to go to the menu causes this error.

comment:5 Changed 6 years ago by gsathya

Here is a branch if that's easier - https://github.com/gsathya/arm/compare/bug7960 :)

comment:6 Changed 6 years ago by atagar

Resolution: fixed
Status: needs_reviewclosed

Patches pushed. Thanks!

Note: See TracTickets for help on using tickets.