NTS: the M ordinate is dropped from geometry SQL literals and from geometries stored in JSON

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#4,093 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@Ahmed7991 y travaille déjà.

Depuis le 4/11/2026.

  • #4120 par @Ahmed7991 — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
85/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
csharp, postgresql
Domaine
database

Piste de recherche

Le bug est dans src/EFCore.PG.NTS/Storage/Internal/NpgsqlGeometryTypeMapping.cs (ligne 94) et NpgsqlJsonGeometryWktReaderWriter.cs (ligne 33), où geometry.AsText() et value.ToText() suppriment M. Remplacez ces appels par un WKTWriter configuré pour produire toutes les ordonnées (X, Y, Z, M). Fait signifie que POINT ZM(1 2 3 4) survit aux littéraux SQL et aux allers-retours JSON, comme indiqué dans le repro.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Environment
PostgreSQL 18.6 with PostGIS 3.6.4 and GEOS 3.14.1 (postgis/postgis:18-3.6), session TimeZone Etc/UTC
.NET 10 runtime 10.0.11, Npgsql.EntityFrameworkCore.PostgreSQL.NetTopologySuite 10.0.3, EF Core 10.0.4
.NET 11 runtime 11.0.0-rc.2, Npgsql.EntityFrameworkCore.PostgreSQL.NetTopologySuite 11.0.0-rc.2, EF Core 11.0.0-rc.2
Also Npgsql 10.0.3 on both lines, NetTopologySuite 2.6.0, Npgsql.NetTopologySuite 10.0.3, macOS arm64

Both lines print the same output. main at 150011d has the same code.

Geometry SQL literals and geometries inside JSON documents are written as WKT with NTS's default writer, which emits X, Y and Z only. A POINT ZM (1 2 3 4) becomes POINT Z(1 2 3).

Repro 
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Logging;
using NetTopologySuite.Geometries;

await using var context = new TestContext();
await context.Database.EnsureDeletedAsync();
await context.Database.EnsureCreatedAsync();

var zm = new Point(new CoordinateZM(1, 2, 3, 4));
context.Rows.Add(new Row { Maybe = null, Doc = new Doc { Location = zm } });
await context.SaveChangesAsync();

// 1. An inlined geometry constant is written as WKT without M
Console.WriteLine($"(Maybe ?? POINT ZM(1 2 3 4)).M: .NET {zm.M}, database {(await context.Rows.Select(r => (double?)(r.Maybe ?? new Point(new CoordinateZM(1, 2, 3, 4))).M).SingleAsync())?.ToString() ?? "NULL"}");

// 2. A geometry inside a JSON column is stored as WKT without M
await using var fresh = new TestContext();
Console.WriteLine($"JSON document Location.M after a round trip: written {zm.M}, read back {(await fresh.Rows.SingleAsync()).Doc.Location.M}");
Console.WriteLine($"Stored document: {await fresh.Database.SqlQueryRaw<string>("SELECT \"Doc\"::text AS \"Value\" FROM \"Rows\"").SingleAsync()}");

public class TestContext : DbContext
{
    public DbSet<Row> Rows => Set<Row>();

    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        => optionsBuilder
            .UseNpgsql(Environment.GetEnvironmentVariable("Test__Npgsql__DefaultConnection"), o => o.UseNetTopologySuite())
            .LogTo(Console.WriteLine, LogLevel.Information)
            .EnableSensitiveDataLogging();

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.HasPostgresExtension("postgis");
        modelBuilder.Entity<Row>().ComplexProperty(r => r.Doc, d => d.ToJson());
    }
}

public class Row
{
    public int Id { get; set; }
    public Point? Maybe { get; set; }
    public Doc Doc { get; set; }
}

public class Doc
{
    public Point Location { get; set; }
}

Output, the same on both lines:

(Maybe ?? POINT ZM(1 2 3 4)).M: .NET 4, database NULL
JSON document Location.M after a round trip: written 4, read back NaN
Stored document: {"Location": "POINT Z(1 2 3)"}

SELECT ST_M(COALESCE(r."Maybe", GEOMETRY 'POINT Z(1 2 3)'))

The literal comes from NpgsqlGeometryTypeMapping.cs#L94 (geometry.AsText()) and the JSON value from NpgsqlJsonGeometryWktReaderWriter.cs#L33 (value.ToText()). A WKTWriter with all ordinates would keep M in both places. #3236 fixed the SRID part of the JSON writer.

Langage dominant
C#
Étoiles
1.8k
Forks
268
Merge moyen
2 h 35 min
PR mergées (30 j)
136

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de npgsql/efcore.pg

Toutes les issues de npgsql/efcore.pg

Issues similaires

Plus d'issues C#

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.