mirror of
https://github.com/bec-project/bec_widgets.git
synced 2025-07-14 11:41:49 +02:00
refactor: improve pylint for WidgetIO
This commit is contained in:
@ -1,3 +1,6 @@
|
|||||||
|
# pylint: disable=no-name-in-module
|
||||||
|
from abc import ABC, abstractmethod
|
||||||
|
|
||||||
from qtpy.QtWidgets import (
|
from qtpy.QtWidgets import (
|
||||||
QApplication,
|
QApplication,
|
||||||
QWidget,
|
QWidget,
|
||||||
@ -12,8 +15,6 @@ from qtpy.QtWidgets import (
|
|||||||
QLabel,
|
QLabel,
|
||||||
)
|
)
|
||||||
|
|
||||||
from abc import ABC, abstractmethod
|
|
||||||
|
|
||||||
|
|
||||||
class WidgetHandler(ABC):
|
class WidgetHandler(ABC):
|
||||||
"""Abstract base class for all widget handlers."""
|
"""Abstract base class for all widget handlers."""
|
||||||
@ -21,15 +22,15 @@ class WidgetHandler(ABC):
|
|||||||
@abstractmethod
|
@abstractmethod
|
||||||
def get_value(self, widget: QWidget):
|
def get_value(self, widget: QWidget):
|
||||||
"""Retrieve value from the widget instance."""
|
"""Retrieve value from the widget instance."""
|
||||||
pass
|
|
||||||
|
|
||||||
@abstractmethod
|
@abstractmethod
|
||||||
def set_value(self, widget: QWidget, value):
|
def set_value(self, widget: QWidget, value):
|
||||||
"""Set a value on the widget instance."""
|
"""Set a value on the widget instance."""
|
||||||
pass
|
|
||||||
|
|
||||||
|
|
||||||
class LineEditHandler(WidgetHandler):
|
class LineEditHandler(WidgetHandler):
|
||||||
|
"""Handler for QLineEdit widgets."""
|
||||||
|
|
||||||
def get_value(self, widget: QLineEdit) -> str:
|
def get_value(self, widget: QLineEdit) -> str:
|
||||||
return widget.text()
|
return widget.text()
|
||||||
|
|
||||||
@ -38,6 +39,8 @@ class LineEditHandler(WidgetHandler):
|
|||||||
|
|
||||||
|
|
||||||
class ComboBoxHandler(WidgetHandler):
|
class ComboBoxHandler(WidgetHandler):
|
||||||
|
"""Handler for QComboBox widgets."""
|
||||||
|
|
||||||
def get_value(self, widget: QComboBox) -> int:
|
def get_value(self, widget: QComboBox) -> int:
|
||||||
return widget.currentIndex()
|
return widget.currentIndex()
|
||||||
|
|
||||||
@ -46,6 +49,8 @@ class ComboBoxHandler(WidgetHandler):
|
|||||||
|
|
||||||
|
|
||||||
class TableWidgetHandler(WidgetHandler):
|
class TableWidgetHandler(WidgetHandler):
|
||||||
|
"""Handler for QTableWidget widgets."""
|
||||||
|
|
||||||
def get_value(self, widget: QTableWidget) -> list:
|
def get_value(self, widget: QTableWidget) -> list:
|
||||||
return [
|
return [
|
||||||
[
|
[
|
||||||
@ -63,6 +68,8 @@ class TableWidgetHandler(WidgetHandler):
|
|||||||
|
|
||||||
|
|
||||||
class SpinBoxHandler(WidgetHandler):
|
class SpinBoxHandler(WidgetHandler):
|
||||||
|
"""Handler for QSpinBox and QDoubleSpinBox widgets."""
|
||||||
|
|
||||||
def get_value(self, widget):
|
def get_value(self, widget):
|
||||||
return widget.value()
|
return widget.value()
|
||||||
|
|
||||||
@ -71,6 +78,8 @@ class SpinBoxHandler(WidgetHandler):
|
|||||||
|
|
||||||
|
|
||||||
class CheckBoxHandler(WidgetHandler):
|
class CheckBoxHandler(WidgetHandler):
|
||||||
|
"""Handler for QCheckBox widgets."""
|
||||||
|
|
||||||
def get_value(self, widget):
|
def get_value(self, widget):
|
||||||
return widget.isChecked()
|
return widget.isChecked()
|
||||||
|
|
||||||
@ -79,6 +88,8 @@ class CheckBoxHandler(WidgetHandler):
|
|||||||
|
|
||||||
|
|
||||||
class LabelHandler(WidgetHandler):
|
class LabelHandler(WidgetHandler):
|
||||||
|
"""Handler for QLabel widgets."""
|
||||||
|
|
||||||
def get_value(self, widget):
|
def get_value(self, widget):
|
||||||
return widget.text()
|
return widget.text()
|
||||||
|
|
||||||
@ -130,7 +141,7 @@ class WidgetIO:
|
|||||||
raise ValueError(f"No handler for widget type: {type(widget)}")
|
raise ValueError(f"No handler for widget type: {type(widget)}")
|
||||||
|
|
||||||
|
|
||||||
##################### Public interface for exporting and importing widget hierarchies #####################
|
################## for exporting and importing widget hierarchies ##################
|
||||||
|
|
||||||
|
|
||||||
class WidgetHierarchy:
|
class WidgetHierarchy:
|
||||||
|
Reference in New Issue
Block a user