Matthew Davis comments

Results 20 comments of


                                            Matthew Davis

alnum() misleading, should use ctype_alnum()

I agree with @webmozart that this is confusing. I would expect that an alphanumeric check would mean that something like `123abc` would also pass because the content of it is...

alnum() misleading, should use ctype_alnum()

@webmozart Of course. This could be fixed and tagged as a different version though? I'd like to contribute to the library to fix issues such as these (I'm going through...

No notInArray() Assertion

@TomasVotruba I can potentially do that over the weekend

inArray / notInArray strict equality inconsistency

This wasn't intentional 😄 Definitely just an oversight!

Conventions for the integer constants in Assert\Assertion

You'll have to check with @beberlei but I don't think there is a specific convention for the numeric values of the assertions. I could be wrong though.

Add a choicesIsset (pair of choicesNotEmpty)

Please feel free to open a PR for this for us to review.

conflict with assets_version

:+1: I'm having this issue also.

conflict with assets_version

To fix: Change this: ``` twig {{ asset('/path/to/your/asset.jpg')|apply_filter('your_filter') }} ``` To: ``` twig {{ asset('/path/to/your/asset.jpg'|apply_filter('your_filter')) }} ```

attribute value 'featured' is always 1

@sujit-neoconcepts Would it be possible to submit a pull request to implement this change please? Thanks

Come up with format for config JSON file

@DanielCoulbourne Maybe we should supplement the `segments` with the concept of Sound Boards. These are collections of pre-recorded audio that can be inserted easily into the final mix down at...