Remove legacy import option (#3536)

* Remove legacy import option

* Always return False in legacy_import_export()
This commit is contained in:
Abdo 2024-11-05 11:25:06 +03:00 committed by GitHub
parent 7629a1b76f
commit 43d9b00b85
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 2 additions and 27 deletions

View file

@ -611,28 +611,6 @@
</layout>
</widget>
</item>
<item>
<widget class="QGroupBox" name="groupBox_4">
<property name="title">
<string>preferences_import_export</string>
</property>
<layout class="QVBoxLayout" name="verticalLayout_11">
<item>
<widget class="QCheckBox" name="legacy_import_export">
<property name="sizePolicy">
<sizepolicy hsizetype="Fixed" vsizetype="Fixed">
<horstretch>0</horstretch>
<verstretch>0</verstretch>
</sizepolicy>
</property>
<property name="text">
<string notr="true">Legacy import/export handling</string>
</property>
</widget>
</item>
</layout>
</widget>
</item>
<item>
<spacer name="verticalSpacer_3">
<property name="orientation">
@ -1293,7 +1271,6 @@
<tabstop>useCurrent</tabstop>
<tabstop>default_search_text</tabstop>
<tabstop>ignore_accents_in_search</tabstop>
<tabstop>legacy_import_export</tabstop>
<tabstop>syncMedia</tabstop>
<tabstop>syncOnProgramOpen</tabstop>
<tabstop>autoSyncMedia</tabstop>

View file

@ -359,7 +359,6 @@ class Preferences(QDialog):
)
self.form.styleLabel.setVisible(not is_win)
self.form.styleComboBox.setVisible(not is_win)
self.form.legacy_import_export.setChecked(self.mw.pm.legacy_import_export())
qconnect(self.form.resetWindowSizes.clicked, self.on_reset_window_sizes)
self.setup_language()
@ -377,8 +376,6 @@ class Preferences(QDialog):
self.mw.pm.setUiScale(newScale)
restart_required = True
self.mw.pm.set_legacy_import_export(self.form.legacy_import_export.isChecked())
if restart_required:
showInfo(tr.preferences_changes_will_take_effect_when_you())

View file

@ -636,7 +636,8 @@ create table if not exists profiles
self.meta[f"{self.editor_key(mode)}TagsCollapsed"] = collapsed
def legacy_import_export(self) -> bool:
return self.meta.get("legacy_import", False)
"Always returns False so users with this option enabled are not stuck on the legacy importer after the UI option is removed."
return False
def set_legacy_import_export(self, enabled: bool) -> None:
self.meta["legacy_import"] = enabled