eiffel-community / eiffel-community/eiffel-pythonlib
Remove context on callbacks
- 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
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