Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an async __unstablePreSavePost hook; resolving with false prevents saving #58022

Merged
merged 12 commits into from
Aug 2, 2024
Merged

Add an async __unstablePreSavePost hook; resolving with false prevents saving #58022

Changes from 4 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
e684c39
Add an async `__unstablePreSavePost` hook; resolving with false preve…
adamsilverstein Jan 19, 2024
f2e65b8
Simplify preSave check
adamsilverstein Jan 19, 2024
101c0d1
Merge branch 'trunk' into add/preSaveHook
adamsilverstein May 8, 2024
bcae174
Filter edits & enable throwing Error in `__unstablePreSavePost` filter
adamsilverstein May 8, 2024
da8196e
Merge branch 'trunk' into add/preSaveHook
adamsilverstein Jul 16, 2024
ff7d6b4
Pass Promise.resolve( edits ) instead of edits
adamsilverstein Jul 16, 2024
4ead1ff
Move REQUEST_POST_UPDATE_START to before preSave async filter
adamsilverstein Jul 16, 2024
d5d7bfc
Update saveEntityRecord error handling to use try/catch
adamsilverstein Jul 16, 2024
a546cdd
Merge branch 'trunk' into add/preSaveHook
adamsilverstein Aug 1, 2024
7764b36
Use explicit false check for error
adamsilverstein Aug 1, 2024
2a0d9f2
reduce changes, test functionality, don’t pass data to filter
adamsilverstein Aug 1, 2024
05b4852
whitespace
adamsilverstein Aug 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 29 additions & 14 deletions packages/editor/src/store/actions.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ export const savePost =
}

const previousRecord = select.getCurrentPost();
const edits = {
let edits = {
id: previousRecord.id,
...registry
.select( coreStore )
Expand All @@ -195,23 +195,38 @@ export const savePost =
),
content,
};
dispatch( { type: 'REQUEST_POST_UPDATE_START', options } );
await registry
.dispatch( coreStore )
.saveEntityRecord(
'postType',
previousRecord.type,
let error;
try {
edits = await applyFilters(
'editor.__unstablePreSavePost',
edits,
options
);
} catch ( err ) {
error = err;
}

let error = registry
.select( coreStore )
.getLastEntitySaveError(
'postType',
previousRecord.type,
previousRecord.id
);
dispatch( { type: 'REQUEST_POST_UPDATE_START', options } );
if ( ! error ) {
error = await registry
.dispatch( coreStore )
.saveEntityRecord(
'postType',
previousRecord.type,
edits,
options
);
}

if ( ! error ) {
error = registry
.select( coreStore )
.getLastEntitySaveError(
'postType',
previousRecord.type,
previousRecord.id
);
}

if ( ! error ) {
await applyFilters(
Expand Down
Loading