David Neilsen comments

Results 42 comments of


                                            David Neilsen

Add a destroy method

This would remove all events from all element, rather than just the one you are selecting.

Error on install: Syntax error or access violation: 1067 Invalid default value for 'end_time'

Or it would be great if the migrations just catered for this.

Fix return types on regex functions

What exactly are the inconsistencies?

Fix return types on regex functions

So it will be ``` $matches = preg_match(...); if (!empty($matches)) { // use $matches } ``` Instead of ``` if (preg_match(..., $matches)) { // use $matches } ```

Fix return types on regex functions

Also for `preg_match_all`, is it no longer going to return the count of matches, and now only the matches array?

Core/Array: in_array

is it supposed to be `in_array` or `array_in`, or even `array_contains`?

Core/Array: extract

Is this even possible to alias? ```

Core/Array: extract

That wont work though, because extract accesses the "current symbol table" which is not available to userland. Unless I'm missing something. Same goes for things like `list`, which is a...

Core/Array: extract

Sure but excluding global scope, this still falls down... https://3v4l.org/ZjrBN

Core/Array: compact

Likewise as per `extract` I don't think this is alias-able.