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

add absolute path #887

Open
wants to merge 763 commits into
base: master
Choose a base branch
from
Open

add absolute path #887

wants to merge 763 commits into from

Conversation

ardianys
Copy link
Contributor

add absolute path to profile picture path, so we can execute script outside src folder.

It raise error if I execute wa script that located outside src folder.

string(18) "pictures/78219.jpg"
PHP Warning:  copy(pictures/78219.jpg): failed to open stream: No such file or directory in /home/xxx/project/whatsapp/php/src/whatsprot.class.php on line 2927

Bashin and others added 30 commits November 30, 2013 15:12
added onGetGroupMessage
added onGetGroupMessage
added onGetGroupMessage
Use ContactSyncV2 instead.
ardianys and others added 25 commits August 12, 2014 19:37
I have no idea where this contacts.php removed to
Now you can send media without downloading the file, just knowing the size and the base64 hash.

TODO: some minor fixes in func.php probably tomorrow
Now broadcast media knowing only size and hash
new ping request from server is like this

```<iq from="s.whatsapp.net" id="1405327577-ping" type="get" xmlns="urn:xmpp:ping"></iq>```
If number add to group
profile picture update notification node now without message node
new group name update notification is like this
                    rx  <notification from="[email protected]" id="894086480" participant="[email protected]" type="subject" t="1407815452" notify="Ardian Yuli">
                    rx    <body>New Group Name</body>
                    rx  </notification>
use pollMessage() instead
FAK U GIT
add absolute path to profile picture path, so we can execute script outside src folder
@ardianys
Copy link
Contributor Author

@shirioko what do you think about this ? I use this feature on my project. Would be nice if my solution merged. thanks.

@mgp25
Copy link
Contributor

mgp25 commented Aug 18, 2014

👍

@hubail hubail force-pushed the master branch 2 times, most recently from d56e5ea to e6cf5a5 Compare May 3, 2015 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.