Opened 21 months ago

Closed 17 months ago

#18566 closed enhancement (fixed)

Expose discussion settings on XML-RPC

Reported by: daniloercoli Owned by: nacin
Priority: normal Milestone: 3.4
Component: XML-RPC Version:
Severity: normal Keywords: mobile has-patch commit
Cc: daniloercoli, kpayne@…, josephscott

Description

To set properly the mobile UI we should be able to access the discussion settings variables from XML-RPC.

  • default_ping_status
  • default_comment_status

Attachments (1)

18566.diff​ (770 bytes) - added by daniloercoli 21 months ago.

Download all attachments as: .zip

Change History (6)

daniloercoli — 21 months ago

comment:1   daniloercoli — 21 months ago

  • Keywords has-patch added; needs-patch removed

@see ​http://blackberry.trac.wordpress.org/ticket/163

comment:2   kurtpayne — 18 months ago

  • Cc kpayne@… added
  • Type changed from defect (bug) to enhancement

comment:3   josephscott — 17 months ago

  • Cc josephscott added
  • Keywords commit added

I'm ok with this idea and patch, with one reservation. The two options default_ping_status & default_comment_status expect string values, specifically: 'open' & 'closed'. Anything other than those two values, from what I've seen so far, will result in it being treated as closed.

Besides that one concern I'm fine with having this patch committed.

comment:4   SergeyBiryukov — 17 months ago

  • Milestone changed from Awaiting Review to 3.4

comment:5   nacin — 17 months ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [19706]:

Expose default_ping_status and default_comment_status via XML-RPC. props daniloercoli. fixes #18566.

Note: See TracTickets for help on using tickets.