← Back to team overview

mahara-contributors team mailing list archive

[Bug 1427031] [NEW] Mobile detect is checking for UA string twice

 

Public bug reported:

In init we call isMobile() twice and that can be expensive. For me it
meant calling Mobile_Detect::match        524 times rather than 262
times when we call it once and remember it's answer.

** Affects: mahara
     Importance: Undecided
         Status: New

-- 
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/1427031

Title:
  Mobile detect is checking for UA string twice

Status in Mahara ePortfolio:
  New

Bug description:
  In init we call isMobile() twice and that can be expensive. For me it
  meant calling Mobile_Detect::match        524 times rather than 262
  times when we call it once and remember it's answer.

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


Follow ups

References