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

Couple of syntax fixes and a cache fix #20

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

Conversation

dwc
Copy link

@dwc dwc commented Jan 25, 2011

Hey Dave,

I'm working with the development version of Mobile Web OSP and have run into some minor issues. These three commits fix the following:

  • There's a minor syntax issue with the iOS detection, introduced in bdc47ff.
  • The LDAP query building for single terms tries to operate on an array when it actually wants a word.
  • Not all systems are set up to have a trailing slash on the temporary directory. The RSS cache breaks (and throws warnings) in this situation, so it's safer to ensure there is a trailing slash.

If you have any questions, please let me know! I'll probably have a few more changes soon.

@dmolsen
Copy link
Owner

dmolsen commented Jan 25, 2011

Nice. Thanks much for the fixes. I'll pull these in tomorrow and will leave the issue open until then.

Daniel Westermann-Clark added 2 commits January 25, 2011 16:56
…re not specfied:

Notice: Undefined index: ua in /home/dwc/src/MIT-Mobile-Web/web/api/index.php on line 38

Notice: Undefined index: callback in /home/dwc/src/MIT-Mobile-Web/web/api/index.php on line 55
Notice: Use of undefined constant nil - assumed 'nil' in /home/dwc/src/MIT-Mobile-Web/web/page_builder/detection.php on line 98
@dwc
Copy link
Author

dwc commented Jan 26, 2011

Hey Dave,

I fixed some additional PHP notices, mostly with using an undefined constant 'nil' where 'NULL' was desired.

Daniel Westermann-Clark added 23 commits January 27, 2011 15:17
fixes issue with calendar module appearing blank on webkit
opposed to the first occurrence in the specified time frame (i.e., the
first gd:when value as used by the convertFeed method).

As a workaround, expand recurring events into single events as
described in the GData Calendar API:

http://code.google.com/apis/calendar/data/2.0/reference.html#Parameters

It's more data to download but it ensures that the events are
correctly sorted for the convertFeed method.
…erwise events will show whatever previous time was given
leads to issues where there is no space betwee the year (e.g. 2011)
and the time (e.g. 11:45 am).
correctly selected on the day view.  Otherwise, in EST you will see
events that started at e.g. 9:00 pm on the previous day.
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