github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2025-04-02 00:24:06 +00:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #17671 from overleaf/mj-project-projection

Browse source 
[web] Use projection when fetching project

GitOrigin-RevId: 4c7eb6582dba9ffa81f8d34c4e353faf1dc15c83
This commit is contained in:
Mathias Jakobsen 2024-03-28 09:53:15 +00:00 committed by Copybot
parent ce87ee3edb
commit 192c497508
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
services/web/app/src/Features/History/HistoryManager.js
View file

@ -3,7 +3,7 @@ const { fetchJson, fetchNothing } = require('@overleaf/fetch-utils')
const settings = require('@overleaf/settings')
const OError = require('@overleaf/o-error')
const UserGetter = require('../User/UserGetter')
const { Project } = require('../../models/Project')
const ProjectGetter = require('../Project/ProjectGetter')
async function initializeProject(projectId) {
const body = await fetchJson(`${settings.apis.project_history.url}/project`, {
@ -115,7 +115,9 @@ async function _deleteProjectInFullProjectHistory(historyId) {
* @returns Promise<object>
*/
async function getCurrentContent(projectId) {
const project = await Project.findById(projectId).exec()
const project = await ProjectGetter.promises.getProject(projectId, {
overleaf: true,
})
const historyId = project?.overleaf?.history?.id
if (!historyId) {
throw new OError('project does not have a history id', { projectId })

8
services/web/test/unit/src/History/HistoryManagerTests.js
View file

@ -48,9 +48,9 @@ describe('HistoryManager', function () {
},
}
this.ProjectModel = {
Project: {
findById: sinon.stub().returns(sinon.stub().resolves(this.project)),
this.ProjectGetter = {
promises: {
getProject: sinon.stub().resolves(this.project),
},
}
@ -59,7 +59,7 @@ describe('HistoryManager', function () {
'@overleaf/fetch-utils': this.FetchUtils,
'@overleaf/settings': this.settings,
'../User/UserGetter': this.UserGetter,
'../../models/Project': this.ProjectModel,
'../Project/ProjectGetter': this.ProjectGetter,
},
})
})