Display Camera Make in PhotoDetails Exif Data

[rschumm]

Lychee is only Displaying «Model» in PhotoDetails Exif Data.
Non-Canon Cameras do not include the Brand Name in the Camera Model - e.g. Sony and Leica will just display the model name. Though, the data is already present in Lychee Backend and Database.
This Feature Displays the Camera Model in a very minimal way.

Note: For Canon Cameara (and maybe others?) the Brand «Canon» will be duplicated, but I think this is ok.

Current Behaviour with a Sony Camera:

Summary by CodeRabbit

• Bug Fixes
  • Improved camera info in photo details: EXIF now shows a cleaner camera string—empty when missing, just the model when make is absent or already included, or "Make - Model" otherwise. The EXIF row still only appears when camera data is present.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d219986b-5fd4-42cd-84d1-066d9695e38c

📥 Commits

Reviewing files that changed from the base of the PR and between 7e2e0fd and b25bdcb.

📒 Files selected for processing (1)

• resources/js/components/drawers/PhotoDetails.vue

🚧 Files skipped from review as they are similar to previous changes (1)

• resources/js/components/drawers/PhotoDetails.vue

---

📝 Walkthrough

Walkthrough

PhotoDetails.vue replaces direct rendering of the preformatted model with a new computed cameraModel that conditionally renders an empty string, the model, or make - model depending on presence and whether the model already includes the make.

Changes

EXIF Camera Display

Layer / File(s)	Summary
cameraModel computed and EXIF UI swap resources/js/components/drawers/PhotoDetails.vue	Imports computed, adds cameraModel (formats make and model with null/empty and prefix rules) and updates the EXIF camera row to display cameraModel instead of the raw preformatted model.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibbled code with care tonight,
Made make and model sit just right,
A dash between, no duplicate name,
The EXIF reads tidy — that's my game!

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[rschumm]

Note: I was not sure if Test is neede for this small feature.

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4bf3f65b-2579-4c47-8887-7df832d88702

📥 Commits

Reviewing files that changed from the base of the PR and between 46c372d and bee34da.

📒 Files selected for processing (1)

• resources/js/components/drawers/PhotoDetails.vue

[ildyria]

Hi, thank you for your PR, I will need to check locally if it gives me "Canon - Canon R5" 👍

[ildyria]

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.23%. Comparing base (46c372d) to head (b25bdcb).
⚠️ Report is 3 commits behind head on master.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[rschumm]

This is wonderful! Thanks a lot for the quick integration and enhancement of my first contribution to Lychee! I love the tool - and your very modern and clean way of working.
more to come from me!