pyrosimple icon indicating copy to clipboard operation
pyrosimple copied to clipboard

Trailing filter after OR statement breaks rtcontrol

Open kannibalox opened this issue 1 year ago • 0 comments

https://github.com/kannibalox/pyrosimple/issues/72#issuecomment-2381024856

Try: rtcontrol is_complete=yes is_open=yes is_ignored=no [ tracker=nothingusesme.net OR tracker=trackerthatexists.net ]

This throws an error. I tried a couple of different things in case I was missing some escaping, but I always get the same error:

username@host:~/bin$ rtcontrol is_complete=yes is_open=yes is_ignored=no [ tracker=\*nothingusesme.net\* OR tracker=\*trackerthatexists.net\* ] fno=1
username@host:~/bin$ rtcontrol is_complete=yes is_open=yes is_ignored=no [ tracker=\*nothingusesme.net\* OR tracker=\*trackerthatexists.net\* ] fno=1
Traceback (most recent call last):
  File "/home/username/test_venv/bin/rtcontrol", line 8, in <module>
    sys.exit(run())
             ^^^^^
  File "/home/username/test_venv/lib/python3.12/site-packages/pyrosimple/scripts/rtcontrol.py", line 1055, in run
    RtorrentControl().run()
  File "/home/username/test_venv/lib/python3.12/site-packages/pyrosimple/scripts/base.py", line 184, in run
    self.mainloop()
  File "/home/username/test_venv/lib/python3.12/site-packages/pyrosimple/scripts/rtcontrol.py", line 787, in mainloop
    self.log.debug("Matcher is: %s", matcher.to_match_string())
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/test_venv/lib/python3.12/site-packages/pyrosimple/util/matching.py", line 193, in to_match_string
    child_strings = [c.to_match_string() for c in self.children]
                     ^^^^^^^^^^^^^^^^^
AttributeError: 'list' object has no attribute 'to_match_string'

kannibalox avatar Sep 29 '24 00:09 kannibalox