Comments for "Services_Nabaztag"

» Submit Your Comment
Comments are only accepted during the "Proposal" phase. This proposal is currently in the "Finished" phase.
» Comments
  • Christian Weiske  [2006-12-09 09:01 UTC]

    - The top level comments should not be indented
    - Use protected - private is not necessary most times
    - You don't handle errors when using "@file()" in apiRequest
    - Make your debug information either XHTML compatible (<br/>), or better use newlines so that it can be used on console, too.

    Apart from this issues, I question the usefulness of the whole thing.
  • Walter Hop  [2006-12-09 15:54 UTC]

    Thanks for the comments Christian.

    First, I would argue that the class is useful. I agree it is not algorithmically complex, but Nabaztag forums and blogs are littered with bad and buggy code snippets, so I see a need for a simple class. The HTTP API is simple to you and me, but perhaps not for the Nabaztag's target group. CPAN and Rubyforge have Nabaztag modules as well.

    A new version of the Nabaztag rabbit is imminent which has more features. It is unclear how Nabaztag.com is going to expose these features in their API but I plan to keep it up-to-date.

    I am in agreement with the style comments and will address these.
  • Christian Weiske  [2006-12-10 12:03 UTC]

    Walter, I don't question the usefulness of your package but the one of the rabbit.
  • Christian Weiske  [2006-12-10 12:03 UTC]

    But that's not the question here
  • Stephan Schmidt  [2006-12-26 00:08 UTC]

    I need this in PEAR. My wife bought me a Nabaztag/tag for X-Mas and I'm desperate to build new services for it.

    BTW: Code looks good.
  • Walter Hop  [2006-12-26 00:19 UTC]

    Hi Stephan, thanks for the positive comment.

    Unfortunately Nabaztag have changed their API when they migrated to their new platform in december. I haven't been able to connect for a week, but yesterday my rabbit came back to life.

    If you pull the class from my SVN, you should have basic functionality again. Api.nabaztag.com (the new url) uses a XML format for return messages which I'd like to parse properly instead of just finding a text, so it might be a few days until I've got everything worked out.
  • Arnaud Limbourg  [2007-12-31 07:35 UTC]

    I know people who would use this !