Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#14651 closed defect (bug) (fixed)

Call to undefined function get_hidden_columns()

Reported by: arena Owned by: scribu
Priority: normal Milestone: 3.1
Component: Administration Version: 3.0.1
Severity: normal Keywords:
Cc: Focuses:

Description

on 3.1 alpha, get_hidden_columns() do not exists anymore ! many plugins will fail when upgrading from 3.0 to 3.1 (which should be a minor update ...)

Change History (21)

#1 @nacin
16 years ago

  • Milestone Awaiting Review3.1
  • Owner set to scribu
  • Severity blockernormal
  • Status newassigned

3.1 is not a minor release.

Assigning to scribu. He'll be rescuing the various functions which were cast away.

#2 follow-up: @scribu
16 years ago

  • Keywords reporter-feedback added

arena: please give examples of such a plugins, so I can see how they're using the function.

#3 in reply to: ↑ 2 @arena
16 years ago

Replying to scribu:

arena: please give examples of such a plugins, so I can see how they're using the function.

MailPress

#4 @scribu
16 years ago

  • Resolutionfixed
  • Status assignedclosed

(In [15514]) Deprecate get_hidden_columns(). Fixes #14651

#5 @nacin
16 years ago

  • Keywords reporter-feedback removed
  • Resolution fixed
  • Status closedreopened

These functions still need to work. We need to bridge all functions removed with the new API without losing any functionality.

#6 @scribu
16 years ago

(In [15517]) Make deprecated functions work. See #14651

#7 @arena
16 years ago

@nacin, any doc on that new api (besides php code) ?

#8 @scribu
16 years ago

Not yet, but you can take a look in these files:

wp-admin/includes/list-table.php
wp-admin/includes/default-list-tables.php

and this ticket: #14579

#9 @arena
16 years ago

last test from svn

bug : Call to undefined function get_hidden_columns() has disappeared

but

screen options still not showing up ! (it did in 3.0)

#10 @scribu
16 years ago

(In [15519]) replace $table with $wp_list_table. See #14651

#11 @scribu
16 years ago

screen options still not showing up ! (it did in 3.0)

That's because it's doing it wrong. If it was calling register_column_headers(), like it was supposed to, it would work (with r15519).

#12 @arena
16 years ago

i'm personnaly using the following filter :

'manage_' . screen . '_columns' ...

#13 follow-up: @scribu
16 years ago

Yes, that filter still exists, but it's meant for modifying already registered columns, like those on the Posts screen.

#14 in reply to: ↑ 13 ; follow-up: @arena
16 years ago

Replying to scribu:

Yes, that filter still exists, but it's meant for modifying already registered columns, like those on the Posts screen.

??? but it worked !!!

#15 follow-ups: @arena
16 years ago

some code consistency with other wp classes

all WP_ classes seems to have their constructor function called

construct()

and are adding (for php4 support) another function with the same name of the class calling $this->construct()

(sample WP_Widget in wp-includes/widgets.php)

#16 in reply to: ↑ 15 @arena
16 years ago

Replying to arena:

some code consistency with other wp classes

all WP_ classes seems to have their constructor function called

construct()

and are adding (for php4 support) another function with the same name of the class calling $this->construct()

(sample WP_Widget in wp-includes/widgets.php)

construct func starts with 2 underscores

#17 in reply to: ↑ 15 ; follow-up: @scribu
16 years ago

all WP_ classes seems to have their constructor function called __construct()

Widgets don't. When we finally drop PHP4, it will be worth renaming them, as you suggest.

#18 in reply to: ↑ 14 @scribu
16 years ago

Replying to arena:

Replying to scribu:

Yes, that filter still exists, but it's meant for modifying already registered columns, like those on the Posts screen.

??? but it worked !!!

All is good then. :)

#19 in reply to: ↑ 17 @arena
16 years ago

Replying to scribu:

all WP_ classes seems to have their constructor function called __construct()

Widgets don't. When we finally drop PHP4, it will be worth renaming them, as you suggest.

widget does see lines 76 and 93 (yes tricky because not in the first lines of the class)

#20 @scribu
16 years ago

  • Resolutionfixed
  • Status reopenedclosed

Closing as fixed. If you notice any other back-compat issues, please open new tickets.

#21 @arena
16 years ago

for info (In [15653]) #14776 undeprecates get_hidden_columns()

Note: See TracTickets for help on using tickets.