Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ISSUE] Saving page : mb_strtolower() expects parameter 1 to be string #4166

Closed
ghost opened this issue Feb 26, 2017 · 6 comments
Closed

[ISSUE] Saving page : mb_strtolower() expects parameter 1 to be string #4166

ghost opened this issue Feb 26, 2017 · 6 comments

Comments

@ghost
Copy link

ghost commented Feb 26, 2017

This is probably connected to my other filed issue:

#4165

##When saving the page i get this error:

mb_strtolower() expects parameter 1 to be string, object given

Refering to line 8 in:
/home/vagrant/Code/pixney.dev/vendor/teamtnt/tntsearch/src/Support/Tokenizer.php

My post data:
_token
"4S1JMkulS2dSqtd9wZkci6piNjYPzLQvBANfZDrL"
page_title_en
"Welcome"
page_slug
"welcome"
entry_content_en
"

Welcome to PyroCMS!

"
page_meta_title_en
""
page_meta_keywords_en
""
page_meta_description_en
""
page_theme_layout
"theme::layouts.homepage"
page_enabled
"on"
page_home
"on"
page_visible
"on"
page_exact
"on"
action
"update"

@ghost
Copy link
Author

ghost commented Feb 26, 2017

Deleting the grid field type assigned to the page type (referred to in the other issue) removed this problem.

@ghost
Copy link
Author

ghost commented Feb 26, 2017

Assigning the same grid field type, but not setting it to translatable. Worked.

@ghost
Copy link
Author

ghost commented Feb 26, 2017

Again assigning the grid field type to the page type but setting it to translatable. Same error appeared!

@ghost
Copy link
Author

ghost commented Feb 26, 2017

I deleted the old grid. Did a new one but didnt set it as translatable. Did a new page field of grid field type and assigned it to home. Set it to translatable.

Same error.

But it seems like this might have solved the other issues issue.

@ghost
Copy link
Author

ghost commented Feb 26, 2017

Finally,

NOT setting the grid translatable
and NOT setting the assigned field to the pagetype translatable threw NO errors.

So there is a bug there..

@RyanThompson
Copy link
Member

This will help prevent this in the future - null was accounted for but not false: laravel-streams/streams-core@671ac8a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant