Skip to content

Renamed variable, code style fixes.

Henrik Pejer requested to merge update-upload into master

I renamed $dontchecksha to $checksha1. Since the variable relates to an action taken, then I think it's easier to read and understand the code if the action isn't a "negative" one. I think that

dontchecksha1 === true

looks weird, since true signals, sortof, forward motion, going forward, then dontchecksha1 === true is not as easy to read as

checksha1 === true.

Mikke, bury this PR if you don't agree, no problem.

Merge request reports