|
|
Movies widget still has letters problem
|
|
|
|
Addicted to MacNN
Join Date: Mar 2006
Status:
Offline
|
|
How hard can it be to get this right?
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Aug 2004
Location: FFM
Status:
Offline
|
|
Originally Posted by peeb
How hard can it be to get this right?
Since the widget is JavaScript it comes with the source code. If it's not hard to fix, just do so yourself.
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Jul 2005
Location: Cooperstown '09
Status:
Offline
|
|
I simply won't put up with this kind of mediocre programming Apple.
|
|
|
|
|
|
|
|
|
Clinically Insane
Join Date: Jun 2001
Location: planning a comeback !
Status:
Offline
|
|
What's a "programming Apple" ?
-t
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Jul 2005
Location: Cooperstown '09
Status:
Offline
|
|
Originally Posted by turtle777
What's a "programming Apple" ?
-t
Smart fruit.
|
|
|
|
|
|
|
|
|
Clinically Insane
Join Date: Jun 2001
Location: planning a comeback !
Status:
Offline
|
|
Originally Posted by rickey939
Smart fruit.
I prefer dumb meat. Pork, beef, chicken. Anything.
-t
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Mar 2006
Status:
Offline
|
|
Originally Posted by TETENAL
Since the widget is JavaScript it comes with the source code. If it's not hard to fix, just do so yourself.
Are you serious? Even if I wanted to do that, what about every other instance of this widget?
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Aug 2004
Location: FFM
Status:
Offline
|
|
Here you go:
Make a copy of the Movies widget into your own ~/Library/Widgets and change line 1201 of ~/Library/Widgets/Movies/Movies.js from
movieObjectForMovieElement.anTheRegex = /^(the|an)\s*(.+)/im;
to
movieObjectForMovieElement.anTheRegex = /^(the |an )\s*(.+)/im;
(
Last edited by TETENAL; May 26, 2008 at 10:27 AM.
Reason: obsolete fix - see below)
|
|
|
|
|
|
|
|
|
Administrator
Join Date: Apr 2001
Location: San Antonio TX USA
Status:
Offline
|
|
It's really just one space that goofs it up? Amazing. That sort of thing is trivial to fix—if you find it and identify that it's a problem. But then, it's trivial for anyone to fix, so until Apple fixes this, why not do it yourself?
One question though: how do you drill down through the widget wrapper to get at the .js file?
|
Glenn -----OTR/L, MOT, Tx
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Aug 2004
Location: FFM
Status:
Offline
|
|
Originally Posted by ghporter
One question though: how do you drill down through the widget wrapper to get at the .js file?
"Show package contents" from either the action menu or the contextual menu.
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Jul 2005
Location: Cooperstown '09
Status:
Offline
|
|
|
|
|
|
|
|
|
|
|
Administrator
Join Date: Apr 2001
Location: San Antonio TX USA
Status:
Offline
|
|
Uh.....I knew that. We'll go with that instead of a really big "duh!" OK? Thanks!
I actually did know how to do this a while back, but I haven't used it and I forgot. I think that may be worse than just not seeing an option on a context menu...
|
Glenn -----OTR/L, MOT, Tx
|
|
|
|
|
|
|
|
Clinically Insane
Join Date: Jun 2001
Location: planning a comeback !
Status:
Offline
|
|
Originally Posted by TETENAL
Here you go:
Make a copy of the Movies widget into your own ~/Library/Widgets and change line 1201 of ~/Library/Widgets/Movies/Movies.js from
movieObjectForMovieElement.anTheRegex = /^(the|an)\s*(.+)/im;
to
movieObjectForMovieElement.anTheRegex = /^(the |an )\s*(.+)/im;
Well, I give you that this is NOT hard to fix once you know what's wrong, and definitely fixable for a programmer, so shame on Apple.
But when you said "If it's not hard to fix, just do so yourself.", did you really think users here would be able to track down the error and fix it themselves ?
I kind a feel stupid now.
-t
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Aug 2004
Location: FFM
Status:
Offline
|
|
OK, I read up how regular expressions actually work, and I think the actual fix for this issue should be: change line 1201 of Movies.js from
movieObjectForMovieElement.anTheRegex = /^(the|an)\s*(.+)/im;
to
movieObjectForMovieElement.anTheRegex = /^(the|an)\s+(.+)/im;
It seems to be a simple mistake. It probably just hasn't been reported to Apple yet.
|
|
|
|
|
|
|
|
|
Clinically Insane
Join Date: Jun 2001
Location: planning a comeback !
Status:
Offline
|
|
Could you please highlight the change. I, uhm, don't see it
Edit: Ok, got it. s+ instead of s*.
-t
|
|
|
|
|
|
|
|
|
Addicted to MacNN
Join Date: Mar 2006
Status:
Offline
|
|
Thanks for figuring out how to fix that - really useful.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Forum Rules
|
|
|
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts
|
HTML code is Off
|
|
|
|
|
|
|
|
|
|
|
|