Opened on 04/13/2015 at 11:10:11 AM

Closed on 04/15/2015 at 09:14:32 AM

Last modified on 05/14/2015 at 02:11:38 PM

#2306 closed change (fixed)

Add TOC macro to web.adblockplus.org

Reported by: kzar Assignee: kzar
Priority: P3 Milestone:
Module: Websites Keywords:
Cc: saroyanm Blocked By:
Blocking: #2118 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: yes
Review URL(s):

http://codereview.adblockplus.org/5158802752012288/

Description

Background

We are working to replace the the anwtoc Anwiki table of contents tags with something that will work after our website has been migrated.

What to change

Create a new toc macro that takes a page name and optional title_tag and outputs the page's TOC HTML based on the result of combination of the get_page_content global function and the toclist filter. Have it be available for include from any template that needs it.

Attachments (0)

Change History (5)

comment:1 Changed on 04/13/2015 at 04:11:05 PM by saroyanm

  • Priority changed from Unknown to P3

comment:2 Changed on 04/13/2015 at 04:11:56 PM by kzar

  • Status changed from new to reviewing

comment:3 Changed on 04/15/2015 at 09:14:32 AM by kzar

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 05/14/2015 at 02:11:38 PM by Ross

  • Verified working set

Implemented and working.

Tested by:
Running the website converter script as per the read me and checking the TOC output for the mentioned pages + serving pages from CMS.

Tested with:
www / r18024
sitescripts / r501
web.adblockplus.org / r68
website-converter / master / 105208883c4336a3788820fe9ab379cf639f8387

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from kzar.
 
Note: See TracTickets for help on using tickets.