Made excel export format enabled by default

[netbrain]

Made excel export format enabled by default
Added PHPExcel as a suggested library through composer

If PHPExcel is not installed, the excel export format will output an error
message describing that PHPExcel is not installed and direct users to read on
documentation website on how to install it.

The error message however is only displayed when someone tries to use the excel format.

Also since composer.json now suggests the phpexcel library, it is given as an option to install when installing semantic result formats.

[JeroenDeDauw]

So someone selecting "excel" from the dropdown on Special:Ask will get an error? It should really not show there unless it can actually be used.

[JeroenDeDauw]

How about you enable it based on a class_exists call?

[netbrain]

Thats exactly what im doing.

The error message only occurs if the required phpexcel library isnt
installed. In which case it tells the user to install the required library
before the format can be used.

This seems less of a hassle for end users then having to install library,
and then enable the library through a php file.
On Mar 14, 2014 4:06 PM, "Jeroen De Dauw" notifications@github.com wrote:

How about you enable it based on a class_exists call?

Reply to this email directly or view it on GitHubhttps://github.com//pull/21#issuecomment-37657022
.

[JeroenDeDauw]

Thats exactly what im doing.

It seems like you are enabling the format regardless and then throwing an error when someone uses it and the library is not loaded. This is quite different from what I suggest, at least feature wise. It's one line of code to add...

[netbrain]

Ah, i understand now. Ill add the excel format only if the class exists.

[JeroenDeDauw]

Continuing in #22.