Skip Menu |
 

This queue is for tickets about the Mac-AppleEvents-Simple CPAN distribution.

Report information
The Basics
Id: 49465
Status: open
Priority: 0/
Queue: Mac-AppleEvents-Simple

People
Owner: cnandor [...] cpan.org
Requestors: melo [...] simplicidade.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.18
Fixed in: (no value)

Attachments
0001-Finder-windows-no-longer-use-the-volume-name-as-titl.patch
0001-Support-Finder.app-with-the-_FXShowPosixPathInTitle-.patch



Subject: Finder windows no longer use volume name
Download (untitled) / with headers
text/plain 171b
Hi, With a recent Mac OS X (10.5.8), a Finder.app window opened on a Volume no longer uses the volume name as the title, but '/'. The attached patch fixes the test case.
Subject: 0001-Finder-windows-no-longer-use-the-volume-name-as-titl.patch
From 2291807718f3187ea5a4cd462d82fb748947892f Mon Sep 17 00:00:00 2001 From: Pedro Melo <melo@simplicidade.org> Date: Sun, 6 Sep 2009 14:48:16 +0100 Subject: [PATCH] Finder windows no longer use the volume name as title (10.5.8) Signed-off-by: Pedro Melo <melo@simplicidade.org> --- t/simple.t | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/t/simple.t b/t/simple.t index 100ce0c..4e363b7 100755 --- a/t/simple.t +++ b/t/simple.t @@ -78,7 +78,7 @@ SKIP: { cmp_ok($!, '==', 0, "No error"); ok(my $get = $evt2->get, "Get value"); - is($get, $volume, "Check value"); + is($get, '/', "Check value"); cmp_ok($!, '==', 0, "No error"); ok(do_event(qw(core clos), $target, "'----':$win"), "Close window"); -- 1.6.4.169.g64d5f
Download (untitled) / with headers
text/plain 465b
On Sun Sep 06 09:50:43 2009, http://www.simplicidade.org/ wrote: Show quoted text
> With a recent Mac OS X (10.5.8), a Finder.app window opened on a > Volume no longer uses the > volume name as the title, but '/'. > > The attached patch fixes the test case.
Odd. I do not get that result at all. Still uses my startup volume name. -- Chris Nandor pudge@pobox.com http://pudge.net/ Slashdot / SourceForge pudge@slashdot.org http://slashdot.org/
Subject: Re: [rt.cpan.org #49465] Finder windows no longer use volume name
Date: Tue, 8 Sep 2009 07:27:48 +0100
To: bug-Mac-AppleEvents-Simple [...] rt.cpan.org
From: Pedro Melo <melo [...] simplicidade.org>
Download (untitled) / with headers
text/plain 743b
Hmms... Might be a preference somewhere? I'll investigate a bit more and if I can find out what it is, I'll re- open the ticket. Thanks, On 2009/09/08, at 06:19, Chris Nandor via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=49465 > > > On Sun Sep 06 09:50:43 2009, http://www.simplicidade.org/ wrote:
>> With a recent Mac OS X (10.5.8), a Finder.app window opened on a >> Volume no longer uses the >> volume name as the title, but '/'. >> >> The attached patch fixes the test case.
> > Odd. I do not get that result at all. Still uses my startup volume > name. > > -- > Chris Nandor pudge@pobox.com http://pudge.net/ > Slashdot / SourceForge pudge@slashdot.org http://slashdot.org/ >
Download (untitled) / with headers
text/plain 446b
On Tue Sep 08 02:28:12 2009, melo@simplicidade.org wrote: Show quoted text
> Hmms... Might be a preference somewhere? > > I'll investigate a bit more and if I can find out what it is, I'll re- > open the ticket.
Found it! http://www.macosxhints.com/article.php?story=20071101210524604 I had changed my defaults to always show the POSIX path. Sorry about that: with the default configuration your tests work. It was my non-documented change. Best regards,
Subject: Re: [rt.cpan.org #49465] Finder windows no longer use volume name
Date: Tue, 8 Sep 2009 08:42:39 -0700
To: bug-Mac-AppleEvents-Simple [...] rt.cpan.org
From: Chris Nandor <cnandor [...] cpan.org>
Download (untitled) / with headers
text/plain 996b
Interesting. I had forgotten that option existed. Maybe I'll do a check for the preference. :-) Or throw it in a TODO block in the test file so it is allowed to fail. Thanks, --Chris On Sep 8, 2009, at 00:54, http://www.simplicidade.org/ via RT wrote: Show quoted text
> Queue: Mac-AppleEvents-Simple > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=49465 > > > On Tue Sep 08 02:28:12 2009, melo@simplicidade.org wrote:
>> Hmms... Might be a preference somewhere? >> >> I'll investigate a bit more and if I can find out what it is, I'll >> re- >> open the ticket.
> > Found it! > > http://www.macosxhints.com/article.php?story=20071101210524604 > > I had changed my defaults to always show the POSIX path. > > Sorry about that: with the default configuration your tests work. It > was my non-documented > change. > > Best regards, >
-- Chris Nandor pudge@pobox.com http://pudge.net/ Slashdot / SourceForge pudge@slashdot.org http://slashdot.org/
Download (untitled) / with headers
text/plain 369b
On Tue Sep 08 11:43:01 2009, CNANDOR wrote: Show quoted text
> Interesting. I had forgotten that option existed. Maybe I'll do a > check for the preference. :-) Or throw it in a TODO block in the > test file so it is allowed to fail.
A TODO block would be great until the check is in place. I'll send a patch later. Do you happen to have an online repo for this distribution?
Subject: Re: [rt.cpan.org #49465] Finder windows no longer use volume name
Date: Tue, 8 Sep 2009 10:09:57 -0700
To: bug-Mac-AppleEvents-Simple [...] rt.cpan.org
From: Chris Nandor <pudge [...] pobox.com>
No need, although if you supply it I won't reject it. You can just provide the patch here. The repo is linked to on http://projects.pudge.net/. It's a SourceForge.net CVS repo. I use git for most things now, so someday it may change. (Maybe now ... I need to look into if SF.net can migrate me automatically.) On Sep 8, 2009, at 09:54, http://www.simplicidade.org/ via RT wrote: Show quoted text
> Queue: Mac-AppleEvents-Simple > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=49465 > > > On Tue Sep 08 11:43:01 2009, CNANDOR wrote:
>> Interesting. I had forgotten that option existed. Maybe I'll do a >> check for the preference. :-) Or throw it in a TODO block in the >> test file so it is allowed to fail.
> > A TODO block would be great until the check is in place. > > I'll send a patch later. > > Do you happen to have an online repo for this distribution?
-- Chris Nandor pudge@pobox.com http://pudge.net/ Slashdot / SourceForge pudge@slashdot.org http://slashdot.org/
Download (untitled) / with headers
text/plain 441b
On Tue Sep 08 12:54:16 2009, http://www.simplicidade.org/ wrote: Show quoted text
> On Tue Sep 08 11:43:01 2009, CNANDOR wrote:
> > Interesting. I had forgotten that option existed. Maybe I'll do a > > check for the preference. :-) Or throw it in a TODO block in the > > test file so it is allowed to fail.
> > A TODO block would be great until the check is in place. > > I'll send a patch later.
Attached a git format-patch file. Best regards,
From ec33757e39bbe38ffb08103a217112aa232f0923 Mon Sep 17 00:00:00 2001 From: Pedro Melo <melo@simplicidade.org> Date: Wed, 9 Sep 2009 12:32:54 +0100 Subject: [PATCH] Support Finder.app with the _FXShowPosixPathInTitle pref set Test 18 of simple.t would fail if _FXShowPosixPathInTitle was set to TRUE, as described in http://www.macosxhints.com/article.php?story=20071101210524604 Signed-off-by: Pedro Melo <melo@simplicidade.org> --- t/simple.t | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/t/simple.t b/t/simple.t index 100ce0c..0a4e7d4 100755 --- a/t/simple.t +++ b/t/simple.t @@ -78,7 +78,10 @@ SKIP: { cmp_ok($!, '==', 0, "No error"); ok(my $get = $evt2->get, "Get value"); - is($get, $volume, "Check value"); + TODO: { + local $TODO = "Fails if you have the _FXShowPosixPathInTitle preference set (see http://bit.ly/165iKR)"; + is($get, $volume, "Check value"); + } cmp_ok($!, '==', 0, "No error"); ok(do_event(qw(core clos), $target, "'----':$win"), "Close window"); -- 1.6.5.rc0


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.