mahara-contributors team mailing list archive

[Robert Lyon <robertl@xxxxxxxxxxxxxxx>]

** Also affects: mahara/1.8
   Importance: Undecided
       Status: New

** Changed in: mahara/1.8
    Milestone: None => 1.8.2

** Changed in: mahara/1.8
     Assignee: (unassigned) => Robert Lyon (robertl-9)

** Changed in: mahara/1.8
   Importance: Undecided => High

** Changed in: mahara/1.8
       Status: New => Confirmed

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1268788

Title:
  mobile_api_json_reply sends extra stuff at the top, making it invalid
  json

Status in Mahara ePortfolio:
  Fix Committed
Status in Mahara 1.8 series:
  Confirmed

Bug description:
  An example is this, using the wrong api token

  D/RestClient( 2917): Response =     <form name="jsdetector_form" id="jsdetector_form" method="post">
  D/RestClient( 2917):         <input name="javascriptenabled" type="hidden" value="true" />
  D/RestClient( 2917):         <script type="text/javascript">
  D/RestClient( 2917):             document.jsdetector_form.submit();
  D/RestClient( 2917):         </script>
  D/RestClient( 2917):     </form>{"fail":"Sorry that mobile upload token was not found. Please check your site and mobile application settings."}

  Stolen from a maharadroid with a patch to log the responses, but that
  can be obtained with easier ways.

  Just need to find the jsdectector stuff and make sure it isn't done in
  a part of mahara that has json reply in it. This may actually break
  some of the json stuff as well? or the /api/mobile/sync.php needs a
  define of JSON at the top.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1268788/+subscriptions