array_push() expects parameter 1 to be array, int given [duplicate]

Question

I am getting array_push() expect parameter one to be an array, any solution?

$activeCourses = array();
foreach ($allCourses as $course) {
   if (strtotime($course->end_date) > time()) {
       $activeCourses = array_push($activeCourses, $course);
   }
}

$activeCourses = array_push($activeCourses, $course); will change the value of $activeCourses to an integer (specifically the number of elements in the array). You just need array_push($activeCourses, $course); to alter the array, or more simply $activeCourses[] = $course;. — kmoser
– kmoser, Commented Nov 3, 2020 at 7:01

zahid hasan emon · Accepted Answer · 2020-11-03 07:05:59Z

1

you have referenced the variable as an array first time. but when you are using it for array push with $activeCourses = it becomes an integer field as array_push returns an integer value and then when it comes to next array push in the next iteration, your activeCourses variable is no longer an array. so use like

$activeCourses = array();
foreach ($allCourses as $course) {
   if (strtotime($course->end_date) > time()) {
       array_push($activeCourses, $course);
   }
}

or

$activeCourses = array();
foreach ($allCourses as $course) {
   if (strtotime($course->end_date) > time()) {
       $activeCourses[] = $course;
   }
}

answered Nov 3, 2020 at 7:05

zahid hasan emon

6,2333 gold badges21 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

bhucho Over a year ago

I would suggest the second method as array_push as an array method consumes both time & memory more

ariefbayu · Accepted Answer · 2020-11-03 07:06:04Z

1

This is caused by you giving $activeCourses the value of array_push(). array_push() return an int value. To fix this, just update this line to not returning value, since array_push()'s parameter is passed by reference:

$activeCourses = array_push($activeCourses, $course);

Changed it to:

array_push($activeCourses, $course);

answered Nov 3, 2020 at 7:06

ariefbayu

22k13 gold badges73 silver badges93 bronze badges

Comments

Burhan Kashour · Accepted Answer · 2020-11-03 06:58:48Z

0

You can use the [] more easier:

$activeCourses = array();
foreach ($allCourses as $course) {
   if (strtotime($course->end_date) > time()) {
     $activeCourses[] = $course;
     }
}

answered Nov 3, 2020 at 6:58

Burhan Kashour

6875 silver badges15 bronze badges

Collectives™ on Stack Overflow

array_push() expects parameter 1 to be array, int given [duplicate]

3 Answers 3

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Linked

Related