Project

General

Profile

Bug #3093

commit 9b91e06 "tvhlog: move to the simple node system" breaks web UI after clicking "Apply configuration (run-time only)"

Added by Heinrich Lech about 9 years ago. Updated about 9 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
User Interface
Target version:
-
Start date:
2015-09-19
Due date:
% Done:

100%

Estimated time:
Found in version:
since 9b91e061f9f2a8a79d4d4ff7ebe619ebb612c517
Affected Versions:

Description

Enabling debug and trace from command line works fine though.

Used git bisect to trace the commit.

History

#1

Updated by Jaroslav Kysela about 9 years ago

I cannot reproduce this here. Which browser ? Any error in the javascript console ?

#2

Updated by Heinrich Lech about 9 years ago

Firefox 40.0.3, I'll have a look to the javascript console.

I also discovered that once clicked the button, the tvheadend process cannot be exited by hitting strg-c in the terminal where I started it nor by the commands "kill" or "killall" (in another terminal of course).

Closing the terminal window (started in the window manager) works though (process is killed then).

#3

Updated by Mark Clarkstone about 9 years ago

Heinrich Lech wrote:

Firefox 40.0.3, I'll have a look to the javascript console.

I also discovered that once clicked the button, the tvheadend process cannot be exited by hitting strg-c in the terminal where I started it nor by the commands "kill" or "killall" (in another terminal of course).

Closing the terminal window (started in the window manager) works though (process is killed then).

Just tried here on my windows machine using Chrome & it works fine here, no errors etc.

Have you tried running Firefox in safe mode to rule out any extension/add-on you have installed?

#4

Updated by Jaroslav Kysela about 9 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Applied in changeset commit:tvheadend|9de43dcf38caa63f270aa69dc69abd9115330eea.

#5

Updated by Jaroslav Kysela about 9 years ago

It wasn't UI related. There was a mutex issue when the debug path was changed. Fixed now.

#6

Updated by Heinrich Lech about 9 years ago

Nice.

Also available in: Atom PDF