Michael Härtl

Results 201 comments of Michael Härtl

> Converting a string obtained in the db driver to a PHP array is risky, the implementation is error-prone. I can not agree to that. The string from a SET...

> The problem is can you implement a deserializer correctly, I mean escaping separators etc, so all edge-cases are handled. If you couldn't then the SET datatype would be broken...

From [here](http://dev.mysql.com/doc/refman/5.1/en/set.html): > A SET is a string object that can have zero or more values, each of which must be chosen from a list of permitted values specified when...

> Ah, should not or cannot contain comma?. So the usage is quite limited? Yeah, but this is is a general limitation of this datatype. Unrelated to the question, whether...

Hmm, this looks more like a problem of the underlying [OAuth2 PHP Server](http://bshaffer.github.io/oauth2-server-php-docs/). I can't really see, how we should fix this. That project uses a standalone request class. Any...

Another option could be, that we set the content on the request object before we use it. Something like this in `ServerComponent::getRequest()`: ``` $this->_request = \OAuth2\Request::createFromGlobals(); $this->_request->content = Yii::app()->getRequest()->getRawData(); ```...

Just fork and create a pull request with your changes. If that's what you meant.

We've seen a similar random error with the below config (see commented line `populateCreatedBy()`). The error suddenly appeared withouth any code change and went away after restarting the server. It...

Sure: ``` org.jooq.exception.SQLDialectNotSupportedException: Type class java.util.UUID is not supported in dialect null at org.jooq.impl.DefaultDataType.getDataType0(DefaultDataType.java:903) at org.jooq.impl.DefaultDataType.getDataType(DefaultDataType.java:835) at org.jooq.impl.DefaultDataType.getDataType(DefaultDataType.java:828) at org.jooq.impl.AbstractField.coerce(AbstractField.java:318) at org.jooq.impl.FieldsImpl.field(FieldsImpl.java:361) at org.jooq.impl.AbstractRow.field(AbstractRow.java:333) at org.jooq.impl.AbstractRecord.field(AbstractRecord.java:200) at com.company.app.config.jooq.DefaultConfiguration$2.populateCreatedBy(DefaultConfiguration.java:63) at com.company.app.config.jooq.DefaultConfiguration$2.insertStart(DefaultConfiguration.java:38)...

Same here. As the PR is open for months and the last commit is from 2021 I wonder if this project is still maintained.