WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#17042 closed defect (bug) (fixed)

Missing closing </fieldset> in user-edit.php for "show admin bar"

Reported by: bluntelk Owned by:
Milestone: 3.2 Priority: normal
Severity: trivial Version: 3.1
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

there is a missing closing </fieldset>tag in user-edit.php in the latest release as well as the latest SVN copy.

I have attached a simple patch to fix the problem

Attachments (2)

patch.17042.txt (712 bytes) - added by bluntelk 5 years ago.
Patch to add in the missing </fieldset> tag
user-edit.php.17042.diff (774 bytes) - added by studionashvegas 5 years ago.
Diff patch - for some reason the text file gave me a "garbage" error…

Download all attachments as: .zip

Change History (16)

@bluntelk
5 years ago

Patch to add in the missing </fieldset> tag

#1 @dd32
5 years ago

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

(In [17593]) Add missing closing fieldset tag. Props bluntelk. Fixes #17042 for trunk

#2 @dd32
5 years ago

  • Milestone changed from Awaiting Review to 3.1.2
  • Resolution fixed deleted
  • Status changed from closed to reopened

Re-opening for 3.1.2 consideration

#3 @ramoonus
5 years ago

don`t tell me its too much work to put this in 3.1.1

#4 @dd32
5 years ago

3.1.1 was in RC status a few days ago, unless it's a security or critical bug, it's not worth delaying the 3.1.1 release.

Version 0, edited 5 years ago by dd32 (next)

#5 @nacin
5 years ago

  • Keywords fixed-major added

This keyword will hide this ticket from reports 5 and 6, as it is fixed for the major release.

#6 @ryanduff
5 years ago

Since this is fixed for 3.2 and 3.1.2 is already out this can probably be closed but I'll let somebody else make that decision.

Honestly, closing this would make one less ticket in the 3.2 milestone.

#7 @nacin
5 years ago

  • Milestone changed from 3.1.2 to 3.1.3

#8 follow-up: @ramoonus
5 years ago

the patch is there
it only includes closing a html tag
less than 5 minutes work ....
no need to close this I think

#9 in reply to: ↑ 8 @aaroncampbell
5 years ago

ramoonus: The real question isn't how much work it is, but rather "Do we want to push another 3.1.x release for this?" Maybe if we need another 3.1 release for something else this can make it in as well. Otherwise I think waiting for 3.2 would be fine.

#10 @ramoonus
5 years ago

i dont think its worth it

#11 @dd32
5 years ago

  • Milestone changed from 3.1.3 to 3.2

I'm closing this as fixed in 3.2, It's not exactly a high priority bug which is going to cause problems for 99% of users; therefor, probably doesn't need a 3.1.x release

@studionashvegas
5 years ago

Diff patch - for some reason the text file gave me a "garbage" error...

#12 @studionashvegas
5 years ago

  • Resolution set to worksforme
  • Status changed from reopened to closed

Tested on 3.2beta1 - new patch made; works just fine.

#13 @nacin
5 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

#14 @nacin
5 years ago

  • Keywords fixed-major removed
  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.