Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
branch: master

Apr 17, 2012

  1. has_key => has_key?

    Change-Id: Iaf10e262cab94f2036992fbcabc7831c29a7c2d5
    Reviewed-on: https://gerrit.causes.com/5611
    Tested-by: Lann Martin <lann@causes.com>
    Reviewed-by: Lann Martin <lann@causes.com>
    Adam Derewecki authored lann committed
  2. responds_to => respond_to

    Change-Id: I807bf14a700a880efb5635e5a54f1df1efd1bc3e
    Reviewed-on: https://gerrit.causes.com/5607
    Tested-by: Adam Derewecki <adam@causes.com>
    Reviewed-by: Adam Derewecki <adam@causes.com>
    Adam Derewecki authored
  3. determine_class doesn't crash on .key? for some inputs

    determine_class was calling .key? on data, and some data inputs don't respond_to
    :key?. This commit gates a call to .has_key? around responds_to :has_key?.
    
    Change-Id: Ia97c8c0973f7e81aaa8d2e204a50cbde9206c6a5
    Reviewed-on: https://gerrit.causes.com/5578
    Reviewed-by: Lann Martin <lann@causes.com>
    Tested-by: Adam Derewecki <adam@causes.com>
    Adam Derewecki authored
  4. Loosen Mogli's hashie dependency

    Once again, for compatibility with our Rails app which already depends
    on a newer version of the gem.
    
    Change-Id: Ib04ed5ad990c92c08878ce3edbb1989142edd99e
    Reviewed-on: https://gerrit.causes.com/5568
    Reviewed-by: Greg Hurrell <greg@causes.com>
    Tested-by: Greg Hurrell <greg@causes.com>
    Greg Hurrell + Alex Wallace authored Greg Hurrell committed
  5. Loosen Mogli's multi_json dependency

    Mogli depended on 1.0.x which is somewhat old (there were two 1.0.x
    releases in 2011, but the gem is already up to 1.3.1).
    
    Update the dependency so that we can use Mogli in our app which already
    depends on a newer version of multi_json.
    
    Change-Id: I23aa83bfd76526c326282f522fad811d5daecebc
    Reviewed-on: https://gerrit.causes.com/5567
    Reviewed-by: Greg Hurrell <greg@causes.com>
    Tested-by: Greg Hurrell <greg@causes.com>
    Greg Hurrell + Alex Wallace authored Greg Hurrell committed

Apr 16, 2012

  1. Mogli is resilient to malformed Facebook responses

    Ok, that title was a little bit of title-baiting. Facebook is happy returning
    purely a number as a valid JSON response, while the RFC for JSON says that a
    valid JSON object must start with a [ or a { (for more info, see:
    http://stackoverflow.com/questions/5034444/can-a-json-start-with). The parser
    Mogli uses had been raising an exception because it assumed it would respond to
    .has_key? if it wasn't already determined to be a list. This commit specifically
    handles the case where the decoded JSON is a Numeric.
    
    Release Notes:
    Related Airbrake Error: https://causes.airbrake.io/errors/43980389
    Related Tracker story: https://www.pivotaltracker.com/story/show/28059743
    
    Change-Id: I08a6f90972297040a40eb1380d24c6788ceaa2f2
    Reviewed-on: https://gerrit.causes.com/5488
    Reviewed-by: Lilia Markham <lilia.markham@causes.com>
    Tested-by: Adam Derewecki <adam@causes.com>
    Adam Derewecki authored
  2. Merge remote-tracking branch 'upstream/master'

    Conflicts:
    	mogli.gemspec
    
    origin/master was the HEAD of master before this merge happened.
    
      *   703f1dd (HEAD, master) Merge remote-tracking branch 'upstream/master'
      |\
      | *   cf14773 (upstream/master) Merge pull request #108 from jonasfa/master
      | |\
      | | * b241486 Adding the 'installed' field to User
      | |/
      | *   9c11855 Merge branch 'master' of github.com:mmangino/mogli
      | |\
      | | *   86dde11 Merge pull request #106 from shwing/master
      | | |\
      | | | * 4101888 daily_active_users to application, likes to photos
      | | |/
      | | *   cd9fc21 Merge pull request #104 from ckbhodge/patch-1
      | | |\
      | | | * b8aa965 added property for 'interested_in' which should be valid:
      https://developers.facebook.com/docs/reference/api/user/
      | | |/
      | | *   79bb3fc Merge pull request #101 from adahmash/master
      | | |\
      | | | * 90d2a14 Added Events assocation to mogli user
      | | | * 9b812aa Upgraded hashtie version for dependency
      | | |/
      | | *   46d0c32 Merge pull request #98 from lecky/master
      | | |\
      | | | * 14cd054 add subscription model
      | | |/
      | * | bccb36b Gemspec change
      | * | 3a0c5aa Just deleted the get_and_map method which causes an
      OAuthException when using create_and_authenticate_as_application for an
      AppRequest. Travis needs to implement it differently for his TestUser class.
      | |/
      | *   fa7b72d Merge pull request #96 from intinig/master
      | |\
      | | * 04f74c7 adding application
      | | * 1a428a6 user likes
      | | * 39646ed Revert "Revert "adding missing fields to comments""
      | | * a776856 Revert "adding missing fields to comments"
      | | * 12cf200 adding missing fields to comments
      | | * f7931c4 adding comments and likes to statuses
      | |/
      | *   d55c3b5 Merge pull request #93 from herval/master
      | |\
      | | * 7da769c friends likes perm
      | | * 4032aa0 publish_actions
      | * |   1ab9b78 Merge pull request #77 from tadruj/patch-1
      | |\ \
      | | * | e29fd7a removed .parsed_response
      | * | |   4795a4e Merge pull request #92 from s1kx/master
      | |\ \ \
      | | * | | 833fbd3 Check Facebook error first before throwing a HTTPException
      | |/ / /
      | * | |   8936d63 Merge branch 'master' of github.com:mmangino/mogli
      | |\ \ \
      | | * \ \   4391368 Merge pull request #89 from jfeng/raise_oauth_error
      | | |\ \ \
      | | | |_|/
      | | |/| |
      | | | * | 70e9114 enable SessionInvalidatedDueToPasswordChange raised
      | | |/ /
      | * | | 2989e21 Clean up profile image code
      | * | |   5f8bc41 Merge branch 'master' of github.com:mmangino/mogli
      | |\ \ \
      | | |/ /
      | | * |   a42bcec Merge pull request #86 from stupied4ever/patch-1
      | | |\ \
      | | | * | 0a55d20 Update lib/mogli/profile.rb
      | | |/ /
      | * | |   c71b52d Merge branch 'master' of github.com:mmangino/mogli
      | |\ \ \
      | | |/ /
      | | * |   91c726c Merge pull request #85 from taweili/master
      | | |\ \
      | | | * | 48e80b6 use HTTMultiParty to support file upload
      | | |/ /
      | * | | c7ac4e1 Add info to readme for creating photos
      | * | | b58af15 merge
      | * | | 1db5371 Bump version
      * | | | 69f6682 (origin/master, origin/HEAD) Store total counts on
      * FetchingArray when fb returns a "count" attribute
      * | | |   f7c5a5c Merge remote-tracking branch 'spidaman/master'
      |\ \ \ \
      | * | | | 6f4f08c (origin/sandbox/richard/mogli) added support for querying
      permissions and support for understanding batch query responses
      | * | | | 6c922f2 bumped version to 0.3.5
      | * | | | 5d3247f added friendlists support
      | * | | | bc2c750 added the rsvp_status attribute to events
      | * | | | b2f6d85 added Place model
      | | |/ /
      | |/| |
    Adam Derewecki authored

Apr 10, 2012

  1. Mike Mangino

    Merge pull request #108 from jonasfa/master

    Adding the 'installed' field to User
    mmangino authored
  2. Jonas Alves

    Adding the 'installed' field to User

    jonasfa authored

Apr 02, 2012

  1. Mike Mangino

    Merge branch 'master' of github.com:mmangino/mogli

    mmangino authored
  2. Mike Mangino

    Gemspec change

    mmangino authored
  3. Mike Mangino

    Merge pull request #106 from shwing/master

    Added missing properties to Application and Photos
    mmangino authored

Apr 01, 2012

  1. Shaun

    daily_active_users to application, likes to photos

    shwing authored

Mar 29, 2012

  1. Mike Mangino

    Merge pull request #104 from ckbhodge/patch-1

    added property for 'interested_in' for User
    mmangino authored
  2. Colin-Kirk Hodge

    added property for 'interested_in' which should be valid: https://dev…

    ckbhodge authored

Feb 23, 2012

  1. Store total counts on FetchingArray when fb returns a "count" attribute

    Previously we were accessing the FetchingArray.count method, which was
    returning the number of entries currently in the array, rather than the
    total number of available entries. When displaying statistics, this
    makes our numbers look small and sad.
    
    When building a fetching array we now store a "count" attribute if
    present in the total_count attribute.
    
    Change-Id: I5dc8f8a629dba727426d151d08e7b79fcbb9ff90
    Reviewed-on: https://gerrit.causes.com/3461
    Reviewed-by: Lilia Markham <lilia.markham@causes.com>
    Tested-by: Lilia Markham <lilia.markham@causes.com>
    Lilia Markham authored

Feb 15, 2012

  1. Mike Mangino

    Merge pull request #101 from adahmash/master

    Added events association to User model
    mmangino authored

Feb 12, 2012

  1. adahmash

    Added Events assocation to mogli user

    adahmash authored
  2. adahmash

    Upgraded hashtie version for dependency

    adahmash authored

Feb 07, 2012

  1. mperice

    Just deleted the get_and_map method which causes an OAuthException wh…

    …en using create_and_authenticate_as_application for an AppRequest. Travis needs to implement it differently for his TestUser class.
    mperice authored mmangino committed

Jan 13, 2012

  1. Mike Mangino

    Merge pull request #98 from lecky/master

    Add subscription model
    mmangino authored
  2. Lecky Lao

    add subscription model

    leckylao authored

Dec 20, 2011

  1. Mike Mangino

    Merge pull request #96 from intinig/master

    Added a lot of missing attributes on Comment / App / Status
    mmangino authored
  2. adding application

    Giovanni Intini authored

Dec 19, 2011

  1. user likes

    Giovanni Intini authored
  2. Revert "Revert "adding missing fields to comments""

    This reverts commit a776856.
    Giovanni Intini authored
  3. Revert "adding missing fields to comments"

    This reverts commit 12cf200.
    Giovanni Intini authored
  4. adding missing fields to comments

    Giovanni Intini authored
  5. adding comments and likes to statuses

    Giovanni Intini authored

Dec 05, 2011

  1. Mike Mangino

    Merge pull request #93 from herval/master

    missing perms added to user class
    mmangino authored
  2. Herval Freire

    friends likes perm

    herval authored

Dec 01, 2011

  1. Mike Mangino

    Merge pull request #77 from tadruj/patch-1

    removed .parsed_response
    mmangino authored

Nov 21, 2011

  1. Merge remote-tracking branch 'spidaman/master'

    Richard Zschech authored
  2. Mike Mangino

    Merge pull request #92 from s1kx/master

    Fix HTTPException to not be thrown on every Facebook error
    mmangino authored

Nov 20, 2011

  1. Check Facebook error first before throwing a HTTPException

    Patrick authored
Something went wrong with that request. Please try again.