Over time I find that I shut down a growing number of forums, but I don't want to delete the nodes inside. So I remove all access and add something like "(CLOSED)" to the name of the forum, but having all those CLOSED forum in the forum list turns into a pain.

Of course I need to see them on the 'admin/structure/forum' page, but not on the 'forum' page.

Comments

salvis created an issue. See original summary.

salvis’s picture

Status: Active » Needs review
StatusFileSize
new2.71 KB

  • salvis committed 4196f94 on 7.x-1.x
    Issue #3083683 by salvis: Hide inaccessible forums from the forum list.
    
salvis’s picture

Status: Needs review » Fixed

This patch also hides forums that are in an inaccessible container.

salvis’s picture

Status: Fixed » Patch (to be ported)

Will need to port this to D8, too.

salvis’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev
mrinalini9’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new2.34 KB

Rerolled patch for 8.x-1.x, please review.

Status: Needs review » Needs work

The last submitted patch, 7: forum_access-hide-inaccessible-forums-3083683-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

naresh_bavaskar’s picture

Assigned: Unassigned » naresh_bavaskar
naresh_bavaskar’s picture

Assigned: naresh_bavaskar » Unassigned
Status: Needs work » Needs review
StatusFileSize
new5.01 KB
new2.61 KB

Fixed coding standards, Please review

naresh_bavaskar’s picture

StatusFileSize
new2.61 KB

fixed interdiff file extension broken.

Status: Needs review » Needs work

The last submitted patch, 10: interdiff-7-10.patch, failed testing. View results

naresh_bavaskar’s picture

Status: Needs work » Needs review
StatusFileSize
new5.01 KB

Status: Needs review » Needs work

The last submitted patch, 13: 3083683-10.patch, failed testing. View results

salvis’s picture

Please review and test!

salvis’s picture

Oh, and thank you for working on this, mrinalini9 and naresh_bavaskar!

salvis’s picture

Status: Needs work » Needs review
salvis’s picture

Status: Needs review » Needs work
StatusFileSize
new3.2 KB

Thank you for the patch, @naresh_bavaskar!

The last chunk doesn't belong here though.

salvis’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work
salvis’s picture

Status: Needs work » Needs review
StatusFileSize
new6.32 KB

Fix a number of bugs, including D9 compatibility.

Status: Needs review » Needs work

  • salvis committed c53996bb on 8.x-1.x
    Issue #3083683 by salvis, naresh_bavaskar: Hide closed forums from the '...
salvis’s picture

Status: Needs work » Needs review
salvis’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.