Make WordPress Core

Opened 17 years ago

Closed 11 years ago

#12145 closed enhancement (maybelater)

Enhance activation process to improve integration with site theme

Reported by: dd32 Owned by: westi
Priority: normal Milestone:
Component: Login and Registration Version:
Severity: normal Keywords: needs-patch
Cc: Focuses: multisite

Description (last modified by westi)

wp-activate.php creates many notices on load due to theme hackery

an example to start:

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\comment-template.php on line 777

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\comment-template.php on line 794

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\general-template.php on line 1588

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\query.php on line 2600
Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\query.php on line 2600

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\post-template.php on line 431

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\query.php on line 2600

Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes\post-template.php on line 1172
class="page page-id- page-parent logged-in">

the problem is, that wp-activate.php attempts to simulate a normal WordPress page, the problem arrises that when the header is included and/or custom code in the theme runs on the header, that the no-existance of the current post id (NULL) causes most of the WordPress functions to complain, as they're expecting a correct post object to be returned.

2 options

  1. Update all API function to move to {{{ if ( ! $_post = get_post(..) ) return false; - This should probably be done anyway
  2. Improve the hackery to allow displaying custom pages such as wp-activate.php more transparently.

Furthur to #2, this might involve creating a page in the database which the API functions can access directly, the activate site page would then become a real page in the eyes of WordPress.

Change History (5)

#1 @wpmuguru
17 years ago

  • Milestone 3.0 → Future Release

Custom signup and activation pages by themes and plugins (ex. BuddyPress does both) are already supported.

We are going to look at revamping the registration process in 3.1 or later. Punting.

Related: #12166

#2 @westi
15 years ago

  • Component Warnings/Notices → Multisite
  • Description modified (diff)
  • Summary wp-activate.php creates many notices on load due to theme hackery → Enhance activation process to improve integration with site theme
  • Type defect (bug) → enhancement

Moving to the Multisite Component as this is more of a Multisite issue than a Warnings/Notice fixup.

And also marking as an Enhancement.

When we address this we should move away from as much theme hackery as possible.

#3 @jeremyfelt
13 years ago

  • Keywords multisite removed

#4 @jeremyfelt
13 years ago

  • Component Multisite → Login and Registration
  • Focuses multisite added

#5 @chriscct7
11 years ago

  • Keywords needs-patch added
  • Milestone Future Release
  • Resolution → maybelater
  • Status new → closed

Closing as maybelater. Complete lack of interest on the feature on the ticket over the last 4 years. Feel free to reopen when more interest re-emerges (particularly if there's a patch)

Note: See TracTickets for help on using tickets.