Skip to content
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

[Bug] View source button not hide from user although view source action is disabled #2396

Closed
Michaelsy opened this issue May 18, 2018 · 1 comment

Comments

@Michaelsy
Copy link
Contributor

Michaelsy commented May 18, 2018

View source button is not hide from user although view source action is disabled

Conditions:

  • Greebo.a
  • View source action: disabled
  • Edit action: enabled
  • Read-only for non-logged-in-users (@all)
  • Logged out

Result

  • View source button is shown whereas its function is disabled.

In "Frusterick Manners.e" it works fine. See also live demos: Greebo.a versus Frusterick Manners.e (obsolete)

HTH - Michael Sy.

@eiroca
Copy link
Contributor

eiroca commented Dec 29, 2018

to fix...
in inc\Menu\Item\Edit.php
change from
}
} else {
if(!actionOK($this->type)) throw new \RuntimeException("action disabled: source");
$params['rev'] = $REV;
$this->type = 'source';
$this->accesskey = 'v';
}

to

        } else {
            if(!actionOK("source")) throw new \RuntimeException("action disabled: source");
            $params['rev'] = $REV;
            $this->type = 'source';
            $this->accesskey = 'v';
        }

eiroca added a commit to eiroca/dokuwiki that referenced this issue Feb 25, 2019
splitbrain added a commit that referenced this issue Mar 13, 2019
splitbrain added a commit that referenced this issue Mar 30, 2019
* master: (22 commits)
  Support "local" subdir when listing the smiley directory
  fix zero ID/NS issue with page_findnearest
  add ID/NS = '0' test against page_findnearest
  fix zero value issue with metadata indexer
  add key/value = '0' test to metadata indexer
  fix zero ID issue with linkwiz and search indexer
  fix zero ID issue with wl() and getID()
  add $ID = '0' test against wl() and getID()
  cookie.js when setValue‘s value parameter is false delete entry
  install.php: respect useacl=0 choice, fixes #2576
  cookie.js add def parameter to getValue()
  cookie.js convert value type to string
  set_doku_pref bugfix, closes #2721
  more definition tests of doku_pref, for #1129
  add tests for get/set_doku_pref #2721
  fix Windows tests for #2702
  use absolute URL in index.php when redirecting to doku.php, fixes #2706
  Adding "recursive_groups" setting for authad-plugin in configuration-manager
  Fix notices when using dw CLI
  Fix issue #2396
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants