Template gets imported as `default`

[faustbrian]

Bug description

We're in the process of testing a migration to the Eloquent driver due to significant performance issues with the amount we have. The Eloquent driver solves all of them but we noticed that a lot of our pages broke. We ended up tracking this down to the YAML not being imported as an exact copy. The JSON in the entries table has the template property set to default instead of the template that was specified in the original file.

YAML (source)

---
id: 981b4627-bb43-45c9-8f1b-f1f5e7eb40e0
origin: 765073ff-6d00-47a3-9a05-d1082db6acaa
image: null
title: Integrationer
textarea: null
template: integrations/index
sections: {  }
seo_title: 'REDACTED'
seo_description: 'REDACTED'
seo_og_image: images/REDACTEDjpeg
updated_by: 9d57592f-8f28-47fa-9ae5-6844892b1483
updated_at: 1718307225
---

JSON (entries table)

{
  "background_color": "beige",
  "title": "REDACTED",
  "updated_by": "0c542901-8050-419f-be8a-3c472c09a66d",
  "textarea": null,
  "hero_type": "hero_with_image",
  "template": "default",
  "image": null,
  "sections": [],
  "seo_title": "REDACTED",
  "seo_description": "REDACTED",
  "seo_og_image": "images/REDACTED.png",
  "add_column": false,
  "hide_when_authenticated": false,
  "parent": "c0c7f893-37e6-47f1-b404-ec3ee0f5299a",
  "__localized_fields": [
    "image",
    "title",
    "textarea",
    "sections"
  ]
}

How to reproduce

Import entries with custom templates.

Logs

Environment

Environment
Application Name: Statamic
Laravel Version: 11.44.1
PHP Version: 8.3.17
Composer Version: 2.8.4
Environment: local
Debug Mode: ENABLED
URL: statamic.test/fi
Maintenance Mode: OFF
Timezone: UTC
Locale: en

Cache
Config: NOT CACHED
Events: NOT CACHED
Routes: NOT CACHED
Views: CACHED

Drivers
Broadcasting: null
Cache: file
Database: sqlite
Logs: stack / single
Mail: log
Queue: sync
Session: file

Livewire
Livewire: v3.6.1

Sentry
Enabled: MISSING DSN
Environment: local
Laravel SDK Version: 4.13.0
PHP SDK Version: 4.10.0
Release: NOT SET
Sample Rate Errors: 100%
Sample Rate Performance Monitoring: NOT SET
Sample Rate Profiling: NOT SET
Send Default PII: DISABLED

Statamic
Addons: 5
Sites: 6 (Eesti, Suomi, Suomi (Englanti), and 3 more)
Stache Watcher: Disabled
Static Caching: Disabled
Version: 5.50.0 PRO

Statamic Addons
aerni/advanced-seo: 2.12.0
marcorieser/statamic-livewire: 4.1.0
mitydigital/sitemapamic: 3.2.0
spatie/statamic-responsive-images: 5.2.2
statamic/eloquent-driver: 4.20.0

Statamic Eloquent Driver
Asset Containers: file
Assets: file
Blueprints: file
Collection Trees: file
Collections: file
Entries: eloquent
Forms: file
Global Sets: file
Global Variables: file
Navigation Trees: file
Navigations: file
Revisions: file
Sites: file
Taxonomies: file
Terms: file
Tokens: file

Installation

Fresh statamic/statamic site via CLI

Additional details

No response

[duncanmcclean]

The JSON in the entries table has the template property set to default instead of the template that was specified in the original file.

template: integrations/index

  "template": "integrations/index",

The template in your example looks correct to me... am I missing something?

[duncanmcclean]

Oh, nevermind, I see what you mean.

[faustbrian]

Ah, my bad. I had copied the JSON from my test database after editing it 😅 I'll test the fix soon to see if that resolves it.

[faustbrian]

@duncanmcclean that seems to have solved the storage issue but when we dump the $page variable on our layout the template seems to be again set to default instead of what was stored in data.template during the import.

Statamic\Structures\Page {[#2039 ▼](https://statamic.test/REDACTED#sf-dump-2000621998-ref22039) // resources/views/components/layout.blade.php
  #tree: Statamic\Structures\CollectionTree {[#1946 ▶](https://statamic.test/REDACTED#sf-dump-2000621998-ref21946)}
  #reference: "981b4627-bb43-45c9-8f1b-f1f5e7eb40e0"
  #route: "{parent_uri}/{slug}"
  #parent: Statamic\Structures\Page {[#1954 ▶](https://statamic.test/REDACTED#sf-dump-2000621998-ref21954)}
  #children: []
  #isRoot: false
  #id: "981b4627-bb43-45c9-8f1b-f1f5e7eb40e0"
  #url: null
  #title: null
  #depth: 1
  #data: []
  #augmentationReferenceKey: null
  #setAugmentationReferenceKey: false
  -absoluteUrl: null
  -absoluteUrlWithoutRedirect: null
  -blueprint: null
  -routeData: array:19 [▼
    "background_color" => "beige"
    "title" => "REDACTED"
    "updated_by" => "0c542901-8050-419f-be8a-3c472c09a66d"
    "textarea" => null
    "hero_type" => "hero_with_image"
    "template" => "default"

[ryanmitchell]

@faustbrian does this help with that? #405

[faustbrian]

@ryanmitchell the data seems to be lost as soon as $entry->syncOriginal() is called.

[faustbrian]

It seems to end up in vendor/statamic/cms/src/Stache/Repositories/EntryRepository.php at Line 95 where it calls $entry->structure()->in($site)->find($entry->id()) and that is where we end up with default as template. Ended up there by dding from vendor/statamic/eloquent-driver/src/Entries/EntryRepository.php in findByUri.

What would cause it to end up there? We imported all of our data into the database and can see it's all there and the Eloquent repository is being used for the queries.

[faustbrian]

@ryanmitchell managed to find the issue and got our page to render properly. If I replace https://github.com/statamic/eloquent-driver/blob/4.x/src/Entries/EntryRepository.php#L42 with return $this->query()->where('site', $site)->where('uri', $uri)->first(); it queries the database and gets the correct data. It currently seems to query the Stache instead of the database.

Should probably still also merge https://github.com/statamic/eloquent-driver/pull/405/files on top of fixing the entry repository.

[ryanmitchell]

I'm not sure that the issue - findByUri will run query() on eloquent query builder as thats a public function

[faustbrian]

The find function just a few lines above also calls return $this->query()->where('id', $id)->first(); instead of the parent. If I don't change it the results seem to always be wrong because findByUri always queries Statamic\Stache\Repositories\EntryRepository instead of the database.