you probably meant
$this->set('newsItems', $newsItems);
^^
On 4 Dez., 16:13, Ryan Schmidt <[email protected]> wrote:
> Let's keep this discussion going where it started: on the Google Group, not
> in private email.
>
> On Dec 4, 2010, at 04:08, NOOOOB wrote:
>
>
>
>
>
>
>
>
>
> > <?php
>
> > class NewsController extends AppController {
> > var $name = 'News';
> > var $uses = array('GoogleNews');
> > function index() {
>
> > $rss = $this->set('news', $this->GoogleNews->find('all'));
> > foreach ($rss as $rssItem) {
> > $data = array(
> > 'News' => $rss['GoogleNews']
> > );
> > }
> > }
> > }
> > ?>
>
> > This is my controler, My model looks like this:
>
> > <?php
>
> > class GoogleNews extends AppModel {
> > var $name ='Feed';
> > var $useTable = false;
> > var $useDbConfig = 'MyDealZ';
> > }
> > ?>
>
> > And when i load index.ctp it views data, but I have this error
>
> > nvalid argument supplied for foreach() [APP\controllers
> > \news_controller.php, line 10]
>
> Looks like the first problem is this line:
>
> $rss = $this->set('news', $this->GoogleNews->find('all'));
>
> You're setting the $rss variable to the return value of the $this->set
> method. Checking the documentation for Controller::set...
>
> http://book.cakephp.org/view/427/set
>
> ...nowhere does this suggest that $this->set provides a return value (and
> looking at the source code in cake/libs/controller/controller.php confirms it
> does not return anything). So it's no surprise your foreach loop complains
> when trying to iterate over an empty variable.
>
> It seems like you should instead write:
>
> // Get the data from Google News
> $newsItems = $this->GoogleNews->find('all');
>
> // Put the news items into the view
> $this->set('news', $newsItems);
>
> // Do other things with the news items
> foreach ($newsItems as $newsItem) {
> ...
>
> }
>
> I renamed $rss to $newsItems since $rss isn't the raw RSS XML data (which is
> what I thought you were talking about initially) but simply an array of
> values, much as you would get from any other data source. If I were
> retrieving news items from a database, I wouldn't call the variable $db
> either.
Check out the new CakePHP Questions site http://cakeqs.org and help others with
their CakePHP related questions.
You received this message because you are subscribed to the Google Groups
"CakePHP" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected] For more options, visit this group at
http://groups.google.com/group/cake-php?hl=en
