Ticket #136 (closed defect: fixed)

Opened 5 years ago

Last modified 3 years ago

PageIndex should not show pages you don't have read access to

Reported by: GiorgosKontopoulos Owned by: unassigned
Priority: normal Milestone: 1.1.6.2
Component: actions Version: 1.1.6.1
Severity: normal Keywords: pageindex read access ACL
Cc:

Description

resolved with just adding a check in before doing anything with each page

foreach ($pages as $page) {

if (!$this->HasAccess("read", $pagetag?)) continue;

I will attach a modified pageindex.php that also gets rid of unecessary <strong> tags on every index letter.

Attachments

pageindex.php Download (1.7 KB) - added by GiorgosKontopoulos 5 years ago.
check if read access, improves on <strong> tags output

Change History

Changed 5 years ago by GiorgosKontopoulos

check if read access, improves on <strong> tags output

Changed 4 years ago by DarTar

  • keywords ACL added
  • milestone set to 1.1.6.2

Changed 4 years ago by DarTar

  • status changed from new to closed
  • resolution set to fixed

I've uploaded your code and added a few other modifications. Thanks!

[54]

Changed 3 years ago by JavaWoman

  • status changed from reopened to closed
  • resolution set to fixed

seems to have been reopened as a result of spam removal; closing again.

Note: See TracTickets for help on using tickets.