Null Check when using ModeledResponse with RPC to return a RECORD in SQL
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Comienza por el manejo de la deserialización de resultados JArray y JObject en ModeledResponse. Reproduce una llamada RPC cuyo resultado SQL sea un registro con todos los campos null y verifica después que ya no se produzca InvalidCastException, mientras que los registros no nulos sigan deserializándose normalmente. En el issue no se indica ninguna ruta de archivo ni de prueba.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Null check when using ModeledResponse with RPC calls on SQL.
I am using RPC calls to do all my data manipulations on the server and not on client side.
The SQL query returns row/s as RECORD if successful and null if failed or no result.
The issue arises when its null, the result is in empty json format, ex- {"user" : null, "id" : null, "points" : null}
and when passed via Json Deserialization, it throws invalidcastexception, since the null value cannot be changed to integer.
What I did is created a custom ModeledResponse class and added a null check, as follows
switch (token)
{
case JArray:
{
//Null Return Check
List<Dictionary<string, object>> data = JsonConvert.DeserializeObject<List<Dictionary<string, object>>>(Content!);
bool isNull = true;
foreach (Dictionary<string,object> item in data)
{
if (!isNull)
break;
foreach (KeyValuePair<string, object> pair in item)
{
if (pair.Value != null)
{
isNull = false;
break;
}
}
}
if (isNull)
break;
//End of Null value check
List<T> deserialized = deserialized = JsonConvert.DeserializeObject<List<T>>(Content!, serializerSettings);
if (deserialized != null)
Models = deserialized;
foreach (T model in Models)
{
model.BaseUrl = baseResponse.ResponseMessage!.RequestMessage.RequestUri.GetInstanceUrl().Replace(model.TableName, "").TrimEnd('/');
model.RequestClientOptions = ClientOptions;
}
break;
}
// A single model has been returned
case JObject:
{
Models.Clear();
//Null Return Check
Dictionary<string, object> data = JsonConvert.DeserializeObject<Dictionary<string, object>>(Content!);
bool isNull = true;
foreach (KeyValuePair<string, object> item in data)
{
if (item.Value != null)
{
isNull = false;
break;
}
}
if (isNull)
break;
//End of Null value check
T obj = JsonConvert.DeserializeObject<T>(Content!, serializerSettings);
if (obj != null)
{
obj.BaseUrl = baseResponse.ResponseMessage!.RequestMessage.RequestUri.GetInstanceUrl().Replace(obj.TableName, "").TrimEnd('/');
obj.RequestClientOptions = ClientOptions;
Models.Add(obj);
}
break;
}
}
If possible, please add the null check in the class.
Thanks
- Lenguaje dominante
- C#
- Estrellas
- 700
- Forks
- 105
- Merge medio
- 1 d 21 h
- PR fusionados (30 d)
- 41
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de supabase/supabase-csharp
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
supabase/supabase-csharp#450 ·
Los mantenedores suelen responder en 1 día
-
bug(gotrue): FailureReason matches english messages instead of error_codePosiblemente ocupada @Tr00d la tomó hace 27 días. Abierto
supabase/supabase-csharp#409 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
OAuth State Error for social providers since Supabase.Gotrue v6.1.0Quizá libre de nuevo @Tr00d la tomó hace 38 días y no hay ningún pull request abierto. Abiertobug
supabase/supabase-csharp#377 · 3 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Postgres Changes filters are validated as anon because SetAuth is applied after channel joinQuizá libre de nuevo @Tr00d la tomó hace 50 días y no hay ningún pull request abierto. Abiertoarea: postgrest bug
supabase/supabase-csharp#345 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
parity(functions): never send new-format API key as Authorization Bearer [from supabase-js]Quizá libre de nuevo @Tr00d la tomó hace 76 días y no hay ningún pull request abierto. Abiertobug parity
supabase/supabase-csharp#279 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de supabase/supabase-csharp
Issues similares
-
Money ExploitsAbiertoS: Untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
project-wayfarer/wayfarer-14#1628 ·
Los mantenedores suelen responder en 3 días
-
:watch: Not Triaged dotnet-target-version
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
copilot documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días
-
untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
dotnet/dotnet-api-docs#13124 ·
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día