Comments

pcambra created an issue. See original summary.

pcambra’s picture

Status: Active » Needs review
StatusFileSize
new1.34 KB
pcambra’s picture

This has caused some errors with GEOS enabled:

Exception: ParseException: Expected word but encountered end of stream in GEOSWKTReader->read() (line 38 of /var/www/html/profiles/farm/modules/contrib/geophp/geoPHP/lib/adapters/WKT.class.php)

From @m.stenta comment: "there is an interesting comment in geofield_field_presave()... i wonder if related:"


    /**
     * Edge case. Currently, Drupal will set a field value to the default value if the current value
     * is empty, even if it's set by the user. This bypasses our validation, and currently non-valid WKB
     * data in geom causes catastrophic failures in entity_load. To compensate, we add the default value
     * in early. When the core issue is fixed, we should drop this code.
     *
     * Geofield Issue: http://drupal.org/node/1886852
     * Core Issue: http://drupal.org/node/1253820
     */

It seems that also is saving empty values when it should not

pcambra’s picture

StatusFileSize
new567 bytes

This should be the correct way, not sure if will be affected by geophp but it's a more correct start

  • m.stenta committed 04117cc on 7.x-1.x authored by pcambra
    Issue #3167416 by pcambra: field_attach_validate not called for log...
m.stenta’s picture

Status: Needs review » Fixed

This seems to be working now! I committed it - thanks @pcambra!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.