-
Notifications
You must be signed in to change notification settings - Fork 79
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.
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
Make parsoid transform follow disable_storage setting. #1337
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
let contentReq; | ||
|
||
if (disabledStorage) { | ||
contentReq = this._getPageBundleFromParsoid(hyper, req); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you confirm that this will fetch a stashed version from core? Otherwise, this could lead to dirty diffs? How are you testing this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabling storage also disabeld stashing, since RESTbase uses the storage mechanism to implement the stash. It doesn't use the Stash in MW.
But that should be a non-issue, since VE doesn't use this API anymore. As far as I know, nothing uses stashing in RESTbase. I have a PR up for removing it entirely.
I think that if this PR gets rebased to current master, CI will work (after fixes on master). |
If disable_storage is set, don't try to load the original HTML from cache for transform operations. Bug: T350219 Change-Id: I33c3baae4f0579121b89f6d913852d6c118ebd56
Change-Id: Id6f41779decdcd3bd2c7afdc7dbae143b7be5877
Other than a nit comment, lets squash and merge. |
Which comment? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you!
If disable_storage is set, don't try to load the original HTML from cache for transform operations.
Bug: T350219
Change-Id: I33c3baae4f0579121b89f6d913852d6c118ebd56