eiffel-community / eiffel-community/eiffel-pythonlib

Remove context on callbacks

Offen
#20 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Python
Sterne
8
Forks
12
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### Description

We always call the callbacks with the event object and the context in which this event was sent.
This is a legacy feature that was used when prototyping the eiffellib and should be removed as it does not serve any really good purpose other than clutter in the callbacks.

Note that we need to start by deprecating it and then remove it.

### Motivation

I have found no need for this feature at all and I don't see any real need for it anywhere else.
If there is a need for something like this, then we should instead add a utility module to eiffellib.
For example a function for fetching a link value from an event (get_link("context")).

### Exemplification

Less unnecessary clutter in callbacks.

### Benefits

Less unnecessary clutter in callbacks.

### Possible Drawbacks

Removing this outright will break a lot of tools that rely on it.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Es werden keine Dateien oder Tests genannt. Beginne damit, zu ermitteln, wo Eiffel-Ereignis-Callbacks das Ereignis und den Kontext empfangen, und verfolge dann die betroffenen Callback-Verwender sowie bestehende Kompatibilitäts- oder Deprecation-Muster. Abgeschlossen ist die Aufgabe, wenn das Legacy-Kontextargument vor seiner Entfernung deprecated ist und die Callback-Schnittstelle sowie die betroffenen Verwender konsistent aktualisiert sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
api, backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.