[bugfix] Fix the arguement order for security function

[feng-tao]

Minor bug fix: the perm and view_menu seems to be in the wrong order.

cc @mistercrunch @hughhhh

[codecov-io]

Codecov Report

Merging #4745 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #4745   +/-   ##
=======================================
  Coverage   72.38%   72.38%           
=======================================
  Files         205      205           
  Lines       15374    15374           
  Branches     1182     1182           
=======================================
  Hits        11128    11128           
  Misses       4243     4243           
  Partials        3        3

Impacted Files	Coverage Δ
superset/security.py	74.21% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6fe11f...a75d56e. Read the comment docs.

[xrmx]

Opportunity to add a test?

[mistercrunch]

I'm wondering if the perms are reversed here somehow, which would make this a backward incompatible change...

[feng-tao]

@xrmx, I could add a test if it is an issue. I hope I am not misunderstand the code...