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

Problem adding widgets: 'The title field is required' form validation message #2

Closed
richlove1 opened this issue Jul 7, 2010 · 7 comments

Comments

@richlove1
Copy link

Get this message when trying to add an article on the RSS feed widget or a Twitter account on the Twitter feed widget

@philsturgeon
Copy link
Member

"add an article"?

And, are you inserting a title? ;-)

@richlove1
Copy link
Author

Apologies for any ambiguity,.....if I add a widget instance (e.g. RSS Feed or Twitter feed) and I DO complete the title field, the form validation comes back saying 'Title field is required'. If I comment out ln272 in application\modules\widgets\libraries\Widgets.php to not return form validation then form submits OK.

Also, feeds and tweets are not showing for me (tried BBC World Cup feed) but this could be a local configuration issue that my localhost can't retrieve feeds maybe?)

@richlove1
Copy link
Author

Looking in to this further (I am fairly new to CodeIgniter and very new to PyroCMS) it looks like you are unsetting some values (including title) from $options on ln86 of modules\widgets\controllers\ajax.php.

Not sure why you are unsetting these values but $options is then passed through to form validation and title has now been unset so looks like this is why failing form validation.

Not sure what appropriate solution is - maybe not to unset title from $options? Not sure why anything is being unset from it.

Rich

@otuyelu
Copy link

otuyelu commented Jul 9, 2010

just installed PyroCMS, and was following the video tutorial, but I alos keep getting the 'title field is required" error. The title field is reset to blank, and all other field data remains intact.

@richlove1
Copy link
Author

As a work around solution you can comment out ln86 of modules\widgets\controllers\ajax.php. I am not sure if that is the desirable long term solution.

Even though I have done that, I am not getting any Twitter feeds showing through for me but have raised that as a separate issue

@ghost
Copy link

ghost commented Jul 11, 2010

I second that fix, it works for me. (commenting line 86)
also, if you have widgets... you cannot select text anymore if that line is not commented out. trying to copy the widget code and it starts dragging the widget around. leaving line 86 commented lets me copy that text now.

fyi - editing widgets now fails because of the commented out line. it gives the same error just on editing the widget instead.

@philsturgeon
Copy link
Member

Fixed in v0.9.9.5.

Drag and drop is nothing to do with server side validation pgarman, stop being a crazy person :)

philsturgeon pushed a commit that referenced this issue Jan 14, 2015
Without this it wouldn't install at all
This issue was closed.
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

3 participants