Opened 3 years ago

Last modified 2 years ago

#14125 accepted enhancement

Seperate out non-editable options in edit site

Reported by: wpmuguru Owned by: sorich87
Priority: normal Milestone: Future Release
Component: Multisite Version:
Severity: normal Keywords: has-patch 2nd-opinion
Cc:

Description

In the edit site screen, blog options which are arrays are shown as SERIALIZED and the textbox is disabled. The attached patch pulls those options out of the options metabox and displays the option name in another metabox below.

Related: #14120

Attachments (2)

14125.diff​ (1.7 KB) - added by wpmuguru 3 years ago.
14125.2.diff​ (1.5 KB) - added by sorich87 3 years ago.

Download all attachments as: .zip

Change History (8)

wpmuguru — 3 years ago

comment:1 follow-up: ↓ 2   wpmuguru — 3 years ago

I think this patch also fixes #14120.

comment:2 in reply to: ↑ 1   wpmuguru — 3 years ago

Replying to wpmuguru:

I think this patch also fixes #14120.

I found another upgrade MU blog to check & this wouldn't fix #14120 because in MU allowed themes was blank by default and isn't in shown as serialized.

comment:3 follow-up: ↓ 4   nacin — 3 years ago

  • Keywords needs-refresh added; has-patch removed
  • Milestone changed from Awaiting Review to 3.1

This looks good in terms of cleaning up that page more. Needs refresh though due to the network admin.

sorich87 — 3 years ago

comment:4 in reply to: ↑ 3   sorich87 — 3 years ago

  • Keywords has-patch 2nd-opinion added; needs-refresh removed
  • Owner set to sorich87
  • Status changed from new to accepted

Replying to nacin:

This looks good in terms of cleaning up that page more. Needs refresh though due to the network admin.

New patch. I was not sure about the look of the new non-editable options table.

comment:5   jane — 3 years ago

Pete Mall should take a look since he's all over the network admin.

comment:6   PeteMall — 2 years ago

  • Milestone changed from 3.1 to Future Release

I'm not sure on the best way to separate out the serialized data yet. Punting for now.

Note: See TracTickets for help on using tickets.