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

SteamID to WebAPI #14

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

SteamID to WebAPI #14

wants to merge 20 commits into from

Conversation

TheSeg
Copy link
Contributor

@TheSeg TheSeg commented Apr 27, 2013

Using WebAPI for almost everything in SteamAPI!

  • Moved to HTTPS. Things are a bit more secure this way.
  • Changed SteamID structure in favor of matching to WebAPI's data
    structure.
  • User Bans info implimented as SteamID->GetPlayerBans structure.
  • Player data implimented as SteamID->GetPlayerSummeries structure
  • New unified function getAvatarUrl() for profile images (old ones
    still exist).
  • NEW isCommunityBanned() function to return Community ban status.
  • Friends using WebAPI. However, strcture slightly changed.
  • Some spelling corrections.

Correcting spelling.
Using WebAPI for almost everything!
* Changed SteamID structure in favor of matching to WebAPI's data
structure.
** User Bans info implimented as SteamID->GetPlayerBans structure.
** Player data implimented as SteamID->GetPlayerSummeries strcutre

* New unified function getAvatarUrl() for profile images (old ones
still exist).
* NEW isCommunityBanned() function to return Community ban status.
* Friends using WebAPI. However, strcture slightly changed.

TODO: Impliment MostPlayedGames, Groups, Owned Games, Play times,
* Better Error handling of initual SteamID load.
* Groups now works with WebAPI.
Added optional function to SteamGame objects to poll game Schema.
// Raw structure dump
$this->GetPlayerSummaries = $profile;
$this->GetPlayerBans = $profileBans;
//$this->GetUserGroupList = $proflieGroups;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't commit commented code.

@koraktor
Copy link
Owner

I decided to cherry-pick the typo fixes and HTTPS change for now. Additionally, I added an optional switch to fallback to plain HTTP.

You might want to rebase on current master.

Thanks so far.

Use WebApi::setSecure() to enable or disable HTTPS. Default is enabled.
@TheSeg TheSeg closed this Apr 30, 2013
@koraktor
Copy link
Owner

Did you close this by mistake or is there a specific reason?

@TheSeg
Copy link
Contributor Author

TheSeg commented Apr 30, 2013

Mistake. :(

@TheSeg TheSeg reopened this Apr 30, 2013
@koraktor
Copy link
Owner

Would you mind rebasing this on master first?

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

Successfully merging this pull request may close these issues.

3 participants