-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Adding repair step to fix missing etags - fixes #12172 #13278
Conversation
@@ -79,7 +80,8 @@ public static function getRepairSteps() { | |||
new RepairMimeTypes(), | |||
new RepairLegacyStorages(\OC::$server->getConfig(), \OC_DB::getConnection()), | |||
new RepairConfig(), | |||
new AssetCache() | |||
new AssetCache(), | |||
new FillETags(\OC_DB::getConnection()) |
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.
add trailing ,
so the next repair step does not requrie changing two lines again?
Refer to this link for build results (access rights to CI server needed): |
Looks good 👍 |
class FillETags extends BasicEmitter implements \OC\RepairStep { | ||
|
||
/** @var \OCP\IConfig */ | ||
protected $config; |
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.
Uneeded?
refs #12502 |
f4b802b
to
7966eee
Compare
Refer to this link for build results (access rights to CI server needed): |
The inspection completed: 8 new issues, 4 updated code elements |
Works as intended. I deleted some etags and run the repair step 👍 |
Adding repair step to fix missing etags - fixes #12172
fixes #12172
@dragotin @PVince81 @icewind1991
From the pure sync algorithm pov it is sufficient to fill the etag with any random string which is not empty.