Last modified: 2013-01-09 01:31:08 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T41633, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 39633 - Export of coordinates to KML -- does not export all coordinates due to bad "offset"
Export of coordinates to KML -- does not export all coordinates due to bad "o...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
SemanticMaps (Other open bugs)
master
PC Windows XP
: Unprioritized major (vote)
: ---
Assigned To: Jeroen De Dauw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-24 21:55 UTC by patelmm79
Modified: 2013-01-09 01:31 UTC (History)
0 users

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description patelmm79 2012-08-24 21:55:24 UTC
When exporting semantic query of a list of coordinates to KML, results are empty if list of values is less than 10.  Somehow, "offset" value is set to 10, and cannot be changed by specifying another "offset" value in the query.

I resolved this issue by adding the following to SM_KMLPrinter.php at line 136 (though I do not code in PHP, so someone should review this):

if ( array_key_exists( 'offset', $this->m_params ) ) {
			$link->setParameter( $this->m_params['offset'], 'offset' );
		}
Comment 1 Jeroen De Dauw 2012-08-25 15:36:00 UTC
Thanks for your report :)

This code no longer exists in the latest version (2.0 alpha) of Semantic Maps as it has been replaced by more generic handling which should not have the reported issue. So assuming this is fixed.
Comment 2 patelmm79 2012-10-31 14:37:16 UTC
Please see http://semantic-mediawiki.org/wiki/Talk:Semantic_Maps#problem_with_KML_export_218 -- I am still experiencing this issue in new version of Semantic Maps
Comment 3 Jeroen De Dauw 2012-11-07 12:49:24 UTC
Just had a look and apparently I was wrong earlier that the code no longer exists. I suspect your earlier patch will still work. You can also just try to remove the getLink method altogether from the KMLPrinter.

Anyway, I will try to have this old code be moved out in the next release.
Comment 4 patelmm79 2012-11-08 15:59:13 UTC
Hello, earlier patch works OK for now.
Comment 5 Jeroen De Dauw 2013-01-09 01:31:08 UTC
It IS gone now :)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links