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

Aperta Adatta ai principianti
  • Qualcosa non va?
#4,093 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@Ahmed7991 ci sta già lavorando.

Dal 4/11/2026.

  • #4120 di @Ahmed7991 — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
85/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
csharp, postgresql
Ambito
database

Direzione di ricerca

Il bug si trova in src/EFCore.PG.NTS/Storage/Internal/NpgsqlGeometryTypeMapping.cs (riga 94) e NpgsqlJsonGeometryWktReaderWriter.cs (riga 33), dove geometry.AsText() e value.ToText() scartano M. Sostituisci questi con un WKTWriter configurato per emettere tutte le ordinate (X, Y, Z, M). Fatto significa che POINT ZM(1 2 3 4) sopravvive sia a letterali SQL che a round-trip JSON, come mostrato nel repro.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
C#
Stelle
1.8k
Fork
268
Merge medio
2h 35m
PR unite (30g)
136

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di npgsql/efcore.pg

Tutte le issue di npgsql/efcore.pg

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.