kepstin/picard forked from metabrainz/picard

The enabled_tagger_scripts_texts() function creates a list of enabled
functions which other code can use directly, abstracting away the checks
to see if tagger script is enabled, and whether each individual script
is enabled (and non-empty).

Fixes PICARD-259 (and associated sub-tickets)

This moves the taggerscript evaluation to happen when a file is matched
(i.e. when a file is added to a track), where it has access to the both
the file metadata *and* the track metadata.

In order to ensure that the titles in the user interface are correct,
this is done by saving an extra copy of the original musicbrainz
metadata in track.orig_metadata. The taggerscript is run as per normal
on track.metadata, but the file matching will use track.orig_metadata
and re-run the taggerscript on the merged data.

InfoDialog: reduce code redundancy, subclassing QWidget

It prevents the image to be vertically truncated

Generic class to save/restore dialog geometry, code simplification

It also adds save/restore geometry to:
- password dialog
- info dialogs (fix PICARD-1093)
- tags from filenames dialog
- edit tags dialog

Each geometry is saved as "geometry_<class_name>" persist option (ByteArray())

Display SSL version string

Example: "SSL OpenSSL 1.0.2g  1 Mar 2016"

Traceback (most recent call last):
  File "./picard/ui/options/scripting.py", line 258, in <lambda>
    list_widget.set_remove_connection(lambda: self.remove_from_list_of_scripts(self.ui.script_list.row(item)))
  File "./picard/ui/options/scripting.py", line 323, in remove_from_list_of_scripts
    item.setSelected(True)
AttributeError: 'NoneType' object has no attribute 'setSelected'

Forgotten in metabrainz#854

PICARD-1204: Read and dump values to QSettings only when necessary

Reading and dumping too often from QSettings causes picard to lock up.
This makes sure we only read on start and dump on change.

Use python3 super function instead of calling by class name

PICARD-1202: only use current item if there's actually one

PICARD-1201: Add -N/--no-restore option to not restore persisted sizes and positions

It prevents following error:
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/picard/ui/metadatabox.py", line 292, in contextMenuEvent
column = item.column()
AttributeError: 'NoneType' object has no attribute 'column'

PICARD-1203: Do not show un-supported tags in UI

It is useful for:
- users having UI issues on startup (too small, too big, out of screen, etc...)
- devs for testing purposes

PICARD-1200: fix Options tree pane resizing

Size policy of both panes plus the minimum width enforced for the tree pane are responsible of the issue

Set horizontal policies to Ignored, and use QSplitter.setSizes() to set a smart default.
After this change, it is possible to move the splitter smoothly.

PICARD-1187: Add DSF file support