JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources

prevents the use of attributes that match core functions

Abierto
#918 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Ruby
Estrellas
2.3k
Forks
546
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

I'm creating a resource that has a method attribute, but due to the way it works now:

  define_method attr do
    @model.public_send(options[:delegate] ? options[:delegate].to_sym : attr)
  end unless method_defined?(attr)

It fails to work for me. I know method is a core ruby function, but it's also the word used to describe HTTP's GET and PUT.

One option is to simply drop the unless. Allow rewriting, but warn.

Another option is to provide an in and out type option. Currently I would describe delegate as the in, where it describes how to take in the data, and I would provide a key option as the out functionality (what the resulting key will look like).

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza en la definición del atributo resource mostrada en el issue e inspecciona cómo method_defined? bloqueaba los accessors generados para nombres como method. Decide si las colisiones deben sobrescribirse con una advertencia o mapearse con una clave de salida explícita, y añade cobertura de regresión para el comportamiento elegido.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
rails, ruby
Área
api, backend
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.