Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#17422 closed defect (bug) (fixed)

Invalid argument supplied for foreach() in wp-admin\credits.php on line 75

Reported by: michaelh's profile michaelh Owned by: nacin's profile nacin
Milestone: 3.2 Priority: normal
Severity: normal Version:
Component: Administration Keywords:
Focuses: Cc:

Description

Getting Invalid argument supplied for foreach() in C:\xampp\htdocs\wp32\wp-admin\credits.php on line 75 error clicking on Credits link in admin footer.

(3.2-beta1-17916)

Attachments (1)

Credits ‹ wp32 test — WordPress_1305313362803.png (125.0 KB) - added by michaelh 14 years ago.
Result of var_dump(get_site_transient('wordpress_credits'));

Download all attachments as: .zip

Change History (9)

#1 @nacin
14 years ago

  • Milestone changed from Awaiting Review to 3.2
  • Owner set to nacin
  • Status changed from new to accepted

#2 @nacin
14 years ago

Could you do a var_dump() of get_site_transient('wordpress_credits');

I can add more defensive checks but I need to know what's wrong.

@michaelh
14 years ago

Result of var_dump(get_site_transient('wordpress_credits'));

#3 @devinreams
14 years ago

I'm seeing the same here, Andrew.

Here's plain text: http://d.pr/IkbJ

#4 @nacin
14 years ago

It would probably help if I deployed my changes to production. :-)

Should work now upon transient refresh.

#5 @devinreams
14 years ago

Right. Having removed the transient, credits.php loads as expected.

As I understand, anyone previously running trunk with this issue will not see it in ~12 hours or so (assuming they don't remove the transient and they've updated).

#6 follow-up: @nacin
14 years ago

credits.php gets cached for a week locally. Not ideal for testing, but no need to fetch data often.

On the other hand, since it's likely to be a rarely accessed page, I can drop it a bit.

#7 in reply to: ↑ 6 @devinreams
14 years ago

Replying to nacin:

credits.php gets cached for a week locally. Not ideal for testing, but no need to fetch data often.

Ah yes, was reading my own timeout.

On the other hand, since it's likely to be a rarely accessed page, I can drop it a bit.

Seems fine as-is. And if if the content doesn't get downloaded there's a fallback message.

Anything remaining? I vote resolving 'worksforme' (caveat: clear credits transient).

#8 @nacin
14 years ago

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

Thanks Devin for the help.

Note: See TracTickets for help on using tickets.