hyperstack-org/hyperstack

a Native::Object in component params can't be retrieved

Open

#417 ouverte le 16 juil. 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (531 stars) (43 forks)batch import
buggood first issue

Description

A Native::Object is a BasicObject and it doesn't respond to "tap". It defines method_missing so Native::Object.new.tap returns nil

At hyperstack/ruby/hyper-component/lib/hyperstack/internal/component/props_wrapper.rb:113 .tap returns nil so fetch_from_cache returns nil instead of the Native::Object.

I suggest to rewrite fetch_from_cache without using tap.

Guide contributeur

Stack technique
ruby
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
RubyBasicObject
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
The issue points to the file `hyperstack/ruby/hyper component/lib/hyperstack/internal/component/props wrapper.rb` at line 113. The current implementation uses `.tap` which fails on `Native::Object` since it is a `BasicObject` that does not respond to `tap`. The fix is to rewrite the `fetch from cache` method without using `tap`, for example by storing the value in a variable and then returning it. Check the comments for any additional context or maintainer input.