SageTV Community  

Go Back   SageTV Community > SageTV Development and Customizations > SageTV v7 Customizations > Batch Metadata Tools
Forum Rules FAQs Community Downloads Today's Posts Search

Notices

Batch Metadata Tools This forums is for discussing the user-created Batch Metadata Tools for SageTV.

Reply
 
Thread Tools Search this Thread Display Modes
  #1  
Old 10-06-2010, 08:33 PM
skiingwiz skiingwiz is offline
Sage Aficionado
 
Join Date: Jan 2005
Posts: 366
Bug in phoenix VFS ViewFactory class

I'm not sure if this is where you want this posted. If not, let me know where and I'll move it. (I didn't think you'd want me posting issues on the sagephoenix site.)

I've been playing around with VFS and the UMB API and I've discovered a bug. Two different sage.phoenix.vfs.views.ViewFactory objects will be considered equal if their Set of tags are equal. (Because ViewFactory uses its parent's implementation of the equals method.) This means that many of the ViewFactory objects returned from phoenix.umb.GetViewFactories() (using the latest vfs.xml on googlecode) will be considered equal, when they are not. I suspect that ViewFactory will need to override equals and incorporate the name in the equality check.

This wreaks havoc if you try to put the result of that API call into anything that uses the equals call. (In my case, a JComboBox led me to this discovery.)

I realize the UMB API really isn't publicly released, but since it's included in the phoenix jar, I figured it was fair game.
Reply With Quote
  #2  
Old 10-07-2010, 05:28 AM
stuckless's Avatar
stuckless stuckless is offline
SageTVaholic
 
Join Date: Oct 2007
Location: London, Ontario, Canada
Posts: 9,713
Thanks, I'll fix that up.

You should know that if you are using any phoenix.jar that is not shipped with the bmt dependency (or listed in the sage7 repository), then you will have issues with the current version of bmt.

While the phoenix.jar is publicly available, it's meant for 'internal' phoenix use, initially. These last few builds of the phoenix jar has undergone some very major restructuring, and I suspect that there is more to come. I can't promise that the next phoenix.jar will not break whatever you are attempting to build. I do think that most of the major changes are behnind us, but I've been doing some frequent updates to the apis. I would think that in a month, the APIs will have calmed down again, but for now, just know, things are changing

In the past I've has people ask if they could use the dynamic menus as well, and while I don't have an issue with it personally, I can't guarantee that the API won't change. The only APIs that I "guarantee" to some degree are the phoenix fanart apis, which are safe to use.

good luck.
Reply With Quote
  #3  
Old 10-08-2010, 07:19 AM
stuckless's Avatar
stuckless stuckless is offline
SageTVaholic
 
Join Date: Oct 2007
Location: London, Ontario, Canada
Posts: 9,713
BTW... since you are using the dev phoenix.jar, I did fix the equals issue, in the latest jar.
Reply With Quote
  #4  
Old 10-08-2010, 02:39 PM
skiingwiz skiingwiz is offline
Sage Aficionado
 
Join Date: Jan 2005
Posts: 366
Quote:
Originally Posted by stuckless View Post
BTW... since you are using the dev phoenix.jar, I did fix the equals issue, in the latest jar.
Thanks! And don't worry, I am not using this on my production system. I am just playing around with it in Eclipse using your (awesome) remote APIs. I only reported this problem because I thought it was something you might run into yourself, not because I expect you to be supporting this.

I am excited about the UMB. Which explains my desire to play around with it.
Reply With Quote
  #5  
Old 10-08-2010, 04:47 PM
stuckless's Avatar
stuckless stuckless is offline
SageTVaholic
 
Join Date: Oct 2007
Location: London, Ontario, Canada
Posts: 9,713
Quote:
Originally Posted by skiingwiz View Post
Thanks! And don't worry, I am not using this on my production system. I am just playing around with it in Eclipse using your (awesome) remote APIs. I only reported this problem because I thought it was something you might run into yourself, not because I expect you to be supporting this.

I am excited about the UMB. Which explains my desire to play around with it.
Just imagine how much more productive you'd be, if you actually had some documentation to go along with the new vfs

If you find issue, then feel free to report them. It's by no means rock solid, but it's getting there.
Reply With Quote
  #6  
Old 10-09-2010, 11:44 AM
skiingwiz skiingwiz is offline
Sage Aficionado
 
Join Date: Jan 2005
Posts: 366
Quote:
Originally Posted by stuckless View Post
Just imagine how much more productive you'd be, if you actually had some documentation to go along with the new vfs
That would be nice! Maybe I'll become familiar enough with it to write some of that documentation for you.
Reply With Quote
  #7  
Old 10-09-2010, 12:38 PM
PLUCKYHD PLUCKYHD is offline
SageTVaholic
 
Join Date: Dec 2007
Posts: 6,257
Quote:
Originally Posted by skiingwiz View Post
That would be nice! Maybe I'll become familiar enough with it to write some of that documentation for you.
I am more interested in what you are coding
Reply With Quote
  #8  
Old 10-09-2010, 07:28 PM
skiingwiz skiingwiz is offline
Sage Aficionado
 
Join Date: Jan 2005
Posts: 366
Quote:
Originally Posted by PLUCKYHD View Post
I am more interested in what you are coding
Nothing exciting yet. I'm building a stand-alone VFS viewer, mostly just as a way to become familiar with the API. After that, I have a project in mind, but I'm hoping Phoenix comes out before I get that far.
Reply With Quote
  #9  
Old 10-09-2010, 09:05 PM
skiingwiz skiingwiz is offline
Sage Aficionado
 
Join Date: Jan 2005
Posts: 366
Quote:
Originally Posted by stuckless View Post
Just imagine how much more productive you'd be, if you actually had some documentation to go along with the new vfs
I have a small request on the documentation topic. Could you build the javadoc with the -use option? I find the Use pages to be very helpful when working with an unfamiliar API. Thanks (in advance).
Reply With Quote
  #10  
Old 10-10-2010, 05:46 AM
stuckless's Avatar
stuckless stuckless is offline
SageTVaholic
 
Join Date: Oct 2007
Location: London, Ontario, Canada
Posts: 9,713
Quote:
Originally Posted by skiingwiz View Post
I have a small request on the documentation topic. Could you build the javadoc with the -use option? I find the Use pages to be very helpful when working with an unfamiliar API. Thanks (in advance).
I've added the flag, when I build a new package, the javadoc should have the "use" pages. In the mean time, I don't know how well you know eclipse, but if you right click on a class and then use "Reference -> Workspace", it will show you a list of where that class is used. (same thing for a method in a class, which I find even more useful)
Reply With Quote
  #11  
Old 10-10-2010, 11:05 AM
skiingwiz skiingwiz is offline
Sage Aficionado
 
Join Date: Jan 2005
Posts: 366
Quote:
Originally Posted by stuckless View Post
I've added the flag, when I build a new package, the javadoc should have the "use" pages. In the mean time, I don't know how well you know eclipse, but if you right click on a class and then use "Reference -> Workspace", it will show you a list of where that class is used. (same thing for a method in a class, which I find even more useful)
Yeah, I use that quiet a bit as well. (Good old ctrl+shift+G) I just happened to be in the javadoc the other day and didn't have eclipse open. Thanks!
Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
VFS in Version 4.1.0 toricred Batch Metadata Tools 8 10-02-2010 10:54 AM
Media Playerplugin class crazymateo SageTV Linux 1 12-16-2006 12:45 PM
What is the Sage TV win class? jominor Hardware Support 4 05-22-2006 08:25 AM


All times are GMT -6. The time now is 01:46 PM.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2023, vBulletin Solutions Inc.
Copyright 2003-2005 SageTV, LLC. All rights reserved.