search

slash2009 / XWMM

XBMC Web Media Manager
Other
56 stars 20 forks source link

Movie set manager doesnt work. #88

Closed un1versal closed 9 years ago

un1versal commented 10 years ago

4.1.4 movie set manager cant really add new set

console spews 

"JSON-RPC request failed." Object { responseXML: null, responseText: "{"id":1,"jsonrpc":"2.0","result":{"limits":{"end":0,"start":0,"total":0}}}", responseStream: null, responseJSON: Object, contentType: "application/json", getResponseHeader: .createResponseObject/<.getResponseHeader(), getAllResponseHeaders: .createResponseObject/<.getAllResponseHeaders(), tId: 5, status: 200, statusText: "OK", 4 more… } Object { jsonrpc: "2.0", method: "VideoLibrary.GetMovieSets", params: Object, id: 1 } ext.ux.xbmc.js:11
reflow: 0ms function .getHeight, ext-all.js line 21
reflow: 0ms function Ext.menu.Menu<.constrainScroll, ext-all.js line 21
reflow: 0ms
reflow: 0ms function Ext.lib.Dom.getXY, ext-base.js line 21
reflow: 0ms function Ext.lib.Dom.getXY, ext-base.js line 21
reflow: 0ms
reflow: 0ms function e.prototype.focus, ext-all.js line 21
reflow: 0ms function Ext.grid.GridView<.handleHdOver, ext-all.js line 21
reflow: 0ms
ReferenceError: editor is not defined movieset.js:63
reflow: 0ms function Ext.grid.GridView<.handleHdOver, ext-all.js line 21
reflow: 0ms

using firefox 30 with NO addons.

un1versal commented 10 years ago

Reopened just for a minor snag. I did a quick 4.1.6 internal testing release to have another go. Cleared cache and disabled all addons on browser.

When adding a new set it works however heres a few things I noticed testing 4.1.6

1) Other details set, should have also NONE as a option., atm you need to write none or delete contets.

2) After adding moves to set, need a full refresh, as it doesnt take straight away, even marking as watched is slower to show up in xwmm,

3) Should need refresh and none should be a permanent option on Other details.

Other details could be named Extra details or better name.

ghost commented 10 years ago

1) I agree however it requires a lot of extra (ugly) code that doesn't provide much functionality. As I mentioned on the other issue the UI needs work. It's on my to do list :smile:

2) Set update fixed on #102, and comments in #101

un1versal commented 10 years ago

1) Isnt really important the comments in #100?

ghost commented 10 years ago

Oops I meant #101 here

un1versal commented 10 years ago

Yes I added those comments in #101 about websockets to #100 as a reference so will close 100 when you are happy with the implementation.