MIT-Mobile-Web icon indicating copy to clipboard operation
MIT-Mobile-Web copied to clipboard
verified publisher icon dmolsen

Couple of syntax fixes and a cache fix

Open dwc opened this issue 14 years ago • 2 comments

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 bdc47ff73324f7f812f0dc35549bcc6f9930d446.
  • 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.

dwc avatar Jan 25 '11 16:01 dwc

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

dmolsen avatar Jan 25 '11 19:01 dmolsen

Hey Dave,

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

dwc avatar Jan 26 '11 21:01 dwc