MantisBT
My View | View Issues | Change Log | Roadmap
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002231VCMILauncherpublic2015-08-16 20:212023-06-27 19:19
ReporterDrake 
Assigned ToWarmonger 
PrioritylowSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformWindowsOSWindows 10 HomeOS Versionbuild 10240
Product Version0.98c 
Target VersionFixed in Version 
Summary0002231: Downloading and refreshing repositories error
DescriptionIt says--
File <unknown> is not a valid JSON file!
At line 40, position 1 warning: Comma expected!

In the VCMI launcher command window

It has something to do with downloading and refreshing repositories.
Steps To ReproduceDownload & refresh repositories
Additional InformationThis occurred ever since I updated from 0.97. I always use a fresh installation of Heroes 3 Complete and remove any old folders of VCMI, like in C:\Users\user\Documents\My Games\vcmi.
TagsNo tags attached.
Attached Filestxt file icon VCMI_Launcher_log.txt [^] (4,771 bytes) 2015-08-16 20:21 [Show Content]
? file icon modSettings.json [^] (4,408 bytes) 2015-08-22 14:28

- Relationships

-  Notes
(0005786)
Drake (reporter)
2015-08-22 14:26

 So, after I learned to edit the json files in C:\Users\user\Documents\My Games\vcmi\config for issue 0002230, I kind of noticed what the VCMI command window meant by line 40. When I changed my graphics to Direct3D in settings.json it told me at line 8 there was a comma expected, but that went away after the second time I ran VCMI. At line 40 in modSettings.json, it begins with Dark Elemental when I 'go to' line 40 in Notepad.
So I can only assume that this has something to do with the Dark Elemental mod. Am I right? I'll attach a copy of my modSettings.json.
(0005787)
AVS (administrator)
2015-08-22 14:35

 This line 40 is from http://download.vcmi.eu/mods/repository/repository.json [^] and should be fixed there. (You can edit downloaded copy in \Documents\My Games\vcmi\downloads\ but it will help only until next repository refresh.

As for settings.json it was you fault - you missed comma when added "driver" field - it shall be between fields.
(0005789)
Drake (reporter)
2015-08-22 14:45

 Ah, so it "literally" means comma lol, I couldn't wrap my mind around that. It automatically added one in settings.json. Thanks again for pointing me in the right direction AVS.
(0005790)
AVS (administrator)
2015-08-22 15:42

 > It automatically added one in settings.json.

This is not exactly true :)

Parser can recover some mark-up errors like missing or extra commas (but it does not add anything). settings.json is "special" file - it loaded at startup and being rewritten on any settings change.
(0005792)
Drake (reporter)
2015-08-22 16:30

 Oh :) well that's a really useful function. modSettings.json and repository.json could totally benefit from a setting like that
(0008654)
Ivan (developer)
2023-06-27 19:19

 Already resolved

- Issue History
Date Modified Username Field Change
2015-08-16 20:21 Drake New Issue
2015-08-16 20:21 Drake Status new => assigned
2015-08-16 20:21 Drake Assigned To => Ivan
2015-08-16 20:21 Drake File Added: VCMI_Launcher_log.txt
2015-08-22 14:26 Drake Note Added: 0005786
2015-08-22 14:28 Drake File Added: modSettings.json
2015-08-22 14:30 AVS Assigned To Ivan => Warmonger
2015-08-22 14:35 AVS Note Added: 0005787
2015-08-22 14:45 Drake Note Added: 0005789
2015-08-22 15:42 AVS Note Added: 0005790
2015-08-22 16:30 Drake Note Added: 0005792
2023-06-27 19:19 Ivan Note Added: 0008654
2023-06-27 19:19 Ivan Status assigned => resolved
2023-06-27 19:19 Ivan Resolution open => fixed

Site | Forums | Wiki | Slack | GitHub

Copyright © 2000 - 2024 MantisBT Team
Hosting provided by DigitalOcean