various: fix mismatches in some action stubs and ResourceMgrTask

[alexgsonic1-pixel]

Changes

action::On::calc_ and action::Off::calc_

• Replaced empty ActionEx::calc_() with isFinishedAS(*mTargetIdx_s, *mSeqBankIdx_s) + setFinished()

action::PlayerStoleOpenBase::calc_

• Replaced stub ActionEx::calc_() with isFinishedAS(0, 0) + setFinished()

ksys::res::ResourceMgrTask::setCompactionStopped

• Changed || to if/else if to fix register allocation ordering (two cbz swapped)

All functions updated from W/m to O in uking_functions.csv.

---

This change is 

[alexgsonic1-pixel]

Done — inlined new_counter as suggested and confirmed it still matches with tools/check. All 4 functions pass.

[Pistonight]

Okay, I see you are still working on decompiling other actions. When you are done please change the PR title again to something that fits better (I changed it for you because I thought you were done :p). Then let me know and I can take another look

[alexgsonic1-pixel]

Sorry about the commit noise - was experimenting with batch decompilation and things got messy. Cleaned up now, just the original 4 functions. Title is fine as-is. Ready for review when you have time!

[Pistonight]

No problem, one last thing, can you squash all of your changes into one commit? This allows us to have a clean merge history