Flip isSP2010 check to isSP2007 to include support for SP2013

Flip isSP2010 check to isSP2007 to include support for SP2013

[ejona, 2013-08-26 21:41:35]

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/sharepoint/SharePointAdaptor.java (left):

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/sharepoint/SharePointAdaptor.java:422: //
Retrieving list of databases failed, but we can continue without it.
Don't delete the first line of this comment.

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/sharepoint/SharePointAdaptor.java
(right):

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/sharepoint/SharePointAdaptor.java:249: private
boolean isSP2007;
The style is to do camel case for everything, even acronyms (so Url instead of
URL). So isSp2007 would be more appropriate.

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/sharepoint/SharePointAdaptor.java:343:
isSP2007 = (version == null || version.startsWith("12"));
Just leave out the version.startsWith(), since it can't ever happen. Go ahead
and keep the note about SP 2010 version and why not add 2013 version to the
comments as well.

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/sharepoint/SiteDataClient.java (right):

https://codereview.appspot.com/13243043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/sharepoint/SiteDataClient.java:271: //the
timeout is n * 60, where n is the number of items you want
s#//the#// the#

[ejona, 2013-08-26 22:28:14]

LGTM