starred/audiobookshelf
1
0
Fork 0
mirror of https://github.com/advplyr/audiobookshelf.git synced 2026-05-30 23:40:40 +02:00
Code Issues 1.0k Packages Projects Releases 10 Wiki Activity

[PR #3779] [MERGED] Refactor LibraryItem to use new model #4081

New Issue
Closed
opened 2026-04-25 00:18:14 +02:00 by adam · 0 comments
adam commented 2026-04-25 00:18:14 +02:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/advplyr/audiobookshelf/pull/3779
Author: @advplyr
Created: 1/3/2025
Status: ✅ Merged
Merged: 1/3/2025
Merged by: @advplyr

Base: master ← Head: refactor-library-item

📝 Commits (6)

  • dd0ebdf Implementing toOld functions for LibraryItem/Book/Podcast
  • 4787e7f Updates to LibraryItemController to use new model
  • eb853d9 Fix LibraryItemController unit test
  • 5cd1410 Remove req.oldLibraryItem usage
  • 12c6f2e Update updateMedia endpoint to use new model
  • 5e8678f Remove unused

📊 Changes

14 files changed (+1080 additions, -421 deletions)

View changed files

📝 server/controllers/LibraryItemController.js (+245 -127)
📝 server/managers/AudioMetadataManager.js (+6 -1)
📝 server/managers/CoverManager.js (+20 -3)
📝 server/managers/CronManager.js (+8 -1)
📝 server/managers/PlaybackSessionManager.js (+3 -3)
📝 server/models/Book.js (+340 -26)
📝 server/models/LibraryItem.js (+180 -49)
📝 server/models/Podcast.js (+211 -19)
📝 server/models/PodcastEpisode.js (+47 -39)
📝 server/objects/LibraryItem.js (+0 -18)
📝 server/objects/mediaTypes/Book.js (+12 -110)
📝 server/objects/mediaTypes/Podcast.js (+0 -17)
📝 server/scanner/AudioFileScanner.js (+4 -4)
📝 test/server/controllers/LibraryItemController.test.js (+4 -4)

📄 Description

Same as https://github.com/advplyr/audiobookshelf/pull/3761 but for the LibraryItem model

This handles endpoints in LibraryItemController

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/advplyr/audiobookshelf/pull/3779 **Author:** [@advplyr](https://github.com/advplyr) **Created:** 1/3/2025 **Status:** ✅ Merged **Merged:** 1/3/2025 **Merged by:** [@advplyr](https://github.com/advplyr) **Base:** `master` ← **Head:** `refactor-library-item` --- ### 📝 Commits (6) - [`dd0ebdf`](https://github.com/advplyr/audiobookshelf/commit/dd0ebdf2d8db0bed7403d2c93e8b358c26dc7893) Implementing toOld functions for LibraryItem/Book/Podcast - [`4787e7f`](https://github.com/advplyr/audiobookshelf/commit/4787e7fdb5f4bc209f978159c6f693d051faabd0) Updates to LibraryItemController to use new model - [`eb853d9`](https://github.com/advplyr/audiobookshelf/commit/eb853d9f0919735c9c0f23663e9b1b195801480c) Fix LibraryItemController unit test - [`5cd1410`](https://github.com/advplyr/audiobookshelf/commit/5cd14108f93b2c7d1aff8d46fbb9ade0c687ab51) Remove req.oldLibraryItem usage - [`12c6f2e`](https://github.com/advplyr/audiobookshelf/commit/12c6f2e9a5b7401d5efd548361ec5d31758981a9) Update updateMedia endpoint to use new model - [`5e8678f`](https://github.com/advplyr/audiobookshelf/commit/5e8678f1ccfcbb29eb57da95bd3756b9aedda7c0) Remove unused ### 📊 Changes **14 files changed** (+1080 additions, -421 deletions) <details> <summary>View changed files</summary> 📝 `server/controllers/LibraryItemController.js` (+245 -127) 📝 `server/managers/AudioMetadataManager.js` (+6 -1) 📝 `server/managers/CoverManager.js` (+20 -3) 📝 `server/managers/CronManager.js` (+8 -1) 📝 `server/managers/PlaybackSessionManager.js` (+3 -3) 📝 `server/models/Book.js` (+340 -26) 📝 `server/models/LibraryItem.js` (+180 -49) 📝 `server/models/Podcast.js` (+211 -19) 📝 `server/models/PodcastEpisode.js` (+47 -39) 📝 `server/objects/LibraryItem.js` (+0 -18) 📝 `server/objects/mediaTypes/Book.js` (+12 -110) 📝 `server/objects/mediaTypes/Podcast.js` (+0 -17) 📝 `server/scanner/AudioFileScanner.js` (+4 -4) 📝 `test/server/controllers/LibraryItemController.test.js` (+4 -4) </details> ### 📄 Description Same as https://github.com/advplyr/audiobookshelf/pull/3761 but for the LibraryItem model This handles endpoints in LibraryItemController --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2026-04-25 00:18:14 +02:00
adam closed this issue 2026-04-25 00:18:14 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
authentication
backlog
bug
chapter editor
config-issue
ebooks
encoding/embedding
enhancement
help wanted
listening sessions & progress
planned
possible plugin
progress sync
pull-request
Mirrored from GitHub Pull Request
 sorting/filtering/searching
unable to reproduce
upload
users & permissions
waiting
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam (Adam Melkus)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/audiobookshelf#4081
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?