Fixing various bugs

[colin99d]

Description

Fixes #3471
Fixes #3469
Fixes #3466
Fixes #3465

Ended up refactoring the screener menu to match how we do things elsewhere. Menu should be simpler now.

How has this been tested?

• Please describe the tests that you ran to verify your changes.
• Provide instructions so we can reproduce.
• Please also list any relevant details for your test configuration.

• Make sure affected commands still run in terminal
  • python terminal.py stocks/load aapl/fa/fraud
  • python terminal.py stocks/load aapl/fa/dupont
  • python terminal.py "stocks/scr/set --preset cheap_dividend/historical"
  • python terminal.py stocks/load amzn/ba/interest
  • python terminal.py /stocks/scr/
    • Tested: overview, valuation, financial, ownership, performance, technical, extensively
• Ensure the SDK still works
  • openbb.fa.fraud("TSLA")
  • openbb.fa.dupont("TSLA")
• Check any related reports
  • python terminal.py reports/equity tsla

Checklist:

• Update our Hugo documentation following these guidelines.
• Update our tests following these guidelines.
• Make sure you are following our CONTRIBUTING guidelines.
• If a feature was added make sure to add it to the corresponding scripts file.

Others

• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.

[Chavithra]

Looks ok overall.

Except:`

2022 Nov 18, 08:52 :) /stocks/scr/ $ set --preset break_out_stocks


2022 Nov 18, 08:53 :) /stocks/scr/ $ historical

Error: local variable 'final_screener' referenced before assignment

Error: not enough values to unpack (expected 3, got 0)