markoheijnen / wp-db-driver

45 stars 12 forks source link

Undefined index length #97

Open franz-josef-kaiser opened 9 years ago

franz-josef-kaiser commented 9 years ago

There're some checks for set array keys missing in strip_invalid_text() in db.php. For me it throws an error in the local dev box. No idea what the actual problem is that's causing the error.

We're running the box on PHP 5.5 with DB Server version: 5.5.45 MySQL Community Server (GPL). Any idea how to debug to the root of the problem so I can add a real fix in a PR instead of just stopping the error from bubbling up by checking isset etc.?

franz-josef-kaiser commented 9 years ago

pinging @markoheijnen

markoheijnen commented 9 years ago

Hey, sorry for the delay. Which WordPress version was this?

franz-josef-kaiser commented 9 years ago

n/p – WP 4.2.4 at this time.

franz-josef-kaiser commented 9 years ago

pinging @markoheijnen

markoheijnen commented 8 years ago

@franz-josef-kaiser Sorry for the delays. Will dive into it this week snd check the code for 4.4

markoheijnen commented 8 years ago

Well, longer delay then I expected. I just did released an update. I haven't put any extra checks but the unit tests of WordPress are passing for 4.2/4.3/4.4.