Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Re: updates_set.php #1

Merged
merged 1 commit into from

2 participants

@zhouyaoji

Jon,

Could you merge my branch? There's an issue with the insertUpdate method.

You have the following:
$response = $oauthapp->insertUpdate(null, $params);

It should be changed to this:
$response = $oauthapp->insertUpdate($params);

My branch has the right use of the insertUpdate method.

This is my first pull request. Be kind. I really don't know what I'm doing, but I am following the GitHub instructions. Ick, technical documentation.

Thanks, buddy!

~Joe

@jcleblanc jcleblanc merged commit 4e11440 into jcleblanc:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 24, 2011
  1. @zhouyaoji
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 php5_sdk/updates_set.php
View
4 php5_sdk/updates_set.php
@@ -23,7 +23,7 @@
'imgWidth' => '44'
);
-$response = $oauthapp->insertUpdate(null, $params);
+$response = $oauthapp->insertUpdate($params);
//check update status. If success, update succeeded
if ($response){
@@ -31,4 +31,4 @@
} else {
echo 'update post failed';
}
-?>
+?>
Something went wrong with that request. Please try again.