Opened 2 years ago

Last modified 2 years ago

#6475 closed defect

Parsing of the log folder in the terms for the feedback is broken — at Version 7

Reported by: marder Owned by:
Priority: Must Have Milestone: Alpha 26
Component: UI – Miscellaneous Keywords:
Cc: Patch:

Description (last modified by Langbart)

image of the falsely parsed path

on svn 26716 with win 10

Reproducible

The feature was added with [21898] and after testing past Alpha versions with VirtualBox. The issue is reproducible for A25 [25860], A24 [24936] and A23b [21935].

Change History (14)

comment:1 by marder, 2 years ago

Description: modified (diff)

comment:2 by marder, 2 years ago

Description: modified (diff)

comment:3 by Langbart, 2 years ago

Is the problem also reproducible for A25 [25860], A24 [24937] and A23 [21946] ?

The configPath comes from Engine.GetUserReportConfigPath(), that was introduced with [21898] by elexis.

Line 
1var g_TermsUserReport = {
2 "TermsAndConditions": {
3 "title": translateWithContext("UserReporter Terms and Conditions", "Terms"),
4 "instruction": translate("Please read and accept the UserReporter Terms and Conditions."),
5 "file": "gui/userreport/Terms_and_Conditions.txt",
6 "termsURL": Engine.ConfigDB_GetValue("user", "userreport.url_terms"),
7 "sprintf": {
8 "logPath": setStringTags(Engine.GetUserReportLogPath(), { "font": "sans-bold-12" }),
9 "configPath": setStringTags(Engine.GetUserReportConfigPath(), { "font": "sans-bold-12" })
10 },
11 "config": "userreport.terms",

Recent changes.

Last edited 2 years ago by Langbart (previous) (diff)

by Langbart, 2 years ago

Attachment: GetUserReportConfigPath.jpg added

comment:4 by Langbart, 2 years ago

Windows uses backslash for the path, on macOS with slash works fine.

by Langbart, 2 years ago

Attachment: path.jpg added

comment:5 by marder, 2 years ago

So just the usual os idiosyncrasys

in reply to:  5 comment:6 by Langbart, 2 years ago

Replying to marder:

So just the usual os idiosyncrasys

You could test if A23 was defective from the beginning or if the problem occurred recently? https://releases.wildfiregames.com/

comment:7 by Langbart, 2 years ago

Description: modified (diff)

by Langbart, 2 years ago

Attachment: a25_term.jpg added

by Langbart, 2 years ago

Attachment: a24_term.jpg added

by Langbart, 2 years ago

Attachment: a23b_term.jpg added

by Langbart, 2 years ago

Attachment: a23_noterm.jpg added

by marder, 2 years ago

Attachment: screenshot0551.png added

image of the falsely parsed path

Note: See TracTickets for help on using tickets.