0 false 2010-01-18T17:04:08-06:00 7462 13 app-will-present-success-alert-sheet-even-if-url-in-plist-is-404 189339 24800 open 2010-02-08T08:04:09-06:00 63029 ctshryock Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/13 If the developer mistypes the URL for his feedback form in the Info.plist or if the server happens to be wonky the user currently is shown a success message saying "Your feedback has been sent.". While technically true .. it's been sent to a black hole never to see the light of day. If the url is unavailable we should alert the user and suggest they send in the feedback via email or simply wait and try again later. Key here is to make sure we don't close the window and erase their message. I don't see an easy way to do this using the current connectionDidFinishLoading: method so it might require more reworking. If the developer mistypes the URL for his feedback form in the Info.plist or if the server happens to be wonky the user currently is shown a success message saying "Your feedback has been sent.". While technically true .. it's been sent to a black hole never to see the light of day. If the url is unavailable we should alert the user and suggest they send in the feedback via email or simply wait and try again later. Key here is to make sure we don't close the window and erase their message. I don't see an easy way to do this using the current connectionDidFinishLoading: method so it might require more reworking. <div><p>If the developer mistypes the URL for his feedback form in the Info.plist or if the server happens to be wonky the user currently is shown a success message saying "Your feedback has been sent.". While technically true .. it's been sent to a black hole never to see the light of day.</p> <p>If the url is unavailable we should alert the user and suggest they send in the feedback via email or simply wait and try again later. Key here is to make sure we don't close the window and erase their message.</p> <p>I don't see an easy way to do this using the current connectionDidFinishLoading: method so it might require more reworking.</p></div> 0 true 2010-01-18T17:19:30-06:00 7462 14 sample-php-script-doesnt-use-name-field-we-capture 0 24800 resolved 2010-01-20T09:44:13-06:00 7462 Mike Zornek Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/14 I noticed in the later versions we now capture the user's name. Sadly however the default sample PHP script doesn't utilize it. I'm not interested in parsing the name for anything that might effect the email header so I use it at the top of the main message body like such: @@@ if (eregi('^[a-zA-Z0-9._-]+@[a-zA-Z0-9-]+\.[a-zA-Z.]{2,5}$', $_REQUEST['email'])) { $email = $_REQUEST['email']; $name = $_REQUEST['name']; } else { $email = 'mike@clickablebliss.com'; $name = 'Unknown Submitter'; } $feedback = $_REQUEST['feedback']; $bundleID = $_REQUEST['bundleID']; $systemProfile = $_REQUEST['systemProfile']; $headers = 'From: ' . $email . "\r\n" . 'Reply-To: ' . $email . "\r\n" . 'X-Mailer: PHP/' . phpversion(); $msg .= "$name writes:\n\n"; $msg .= "$feedback\n"; @@@ I noticed in the later versions we now capture the user's name. Sadly however the default sample PHP script doesn't utilize it. I'm not interested in parsing the name for anything that might effect the email header so I use it at the top of the main message body like such: @@@ if (eregi('^[a-zA-Z0-9._-]+@[a-zA-Z0-9-]+\.[a-zA-Z.]{2,5}$', $_REQUEST['email'])) { $email = $_REQUEST['email']; $name = $_REQUEST['name']; } else { $email = 'mike@clickablebliss.com'; $name = 'Unknown Submitter'; } $feedback = $_REQUEST['feedback']; $bundleID = $_REQUEST['bundleID']; $systemProfile = $_REQUEST['systemProfile']; $headers = 'From: ' . $email . "\r\n" . 'Reply-To: ' . $email . "\r\n" . 'X-Mailer: PHP/' . phpversion(); $msg .= "$name writes:\n\n"; $msg .= "$feedback\n"; @@@ <div><p>I noticed in the later versions we now capture the user's name. Sadly however the default sample PHP script doesn't utilize it.</p> <p>I'm not interested in parsing the name for anything that might effect the email header so I use it at the top of the main message body like such:</p> <pre> <code>if (eregi('^[a-zA-Z0-9._-]+@[a-zA-Z0-9-]+\.[a-zA-Z.]{2,5}$', $_REQUEST['email'])) { $email = $_REQUEST['email']; $name = $_REQUEST['name']; } else { $email = 'mike@clickablebliss.com'; $name = 'Unknown Submitter'; } $feedback = $_REQUEST['feedback']; $bundleID = $_REQUEST['bundleID']; $systemProfile = $_REQUEST['systemProfile']; $headers = 'From: ' . $email . "\r\n" . 'Reply-To: ' . $email . "\r\n" . 'X-Mailer: PHP/' . phpversion(); $msg .= "$name writes:\n\n"; $msg .= "$feedback\n";</code> </pre></div> 0 false 2010-01-18T16:37:43-06:00 7462 12 need-to-add-namespace-to-category-methods-appendstring-and-friends 0 24800 open 2010-01-19T13:57:50-06:00 7462 Mike Zornek Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/12 In my own project I had a hell of a time debugging a broken POST body and found out the source to be the following category namespace collision. In NSURLRequest+postForm.m we add some category methods on to NSMutableData @@@ objectivec @interface NSMutableData (append) - (void)appendString:(NSString*)string; - (void)appendFormat:(NSString *)format, ...; @end @@@ Well it seems in 10.6 Apple has some similarly named methods that are colliding with ours. objc[205]: REPLACED: -[NSMutableData appendString:] by category MobileMeNSMutableDataAdditions (IMP was 0x8ac32 (/Users/zorn/Builds/Debug/ProfitTrain.app/Contents/MacOS/ProfitTrain), now 0x92ce23bd (/System/Library/PrivateFrameworks/DotMacSyncManager.framework/Versions/A/DotMacSyncManager)) I don't load the DotMacSyncManager framework and you won't see this collision with a generic new Cocoa app (like the demo app which works fine in 10.6). I assume it's coming along for the ride with WebKit or something I do link against. Anyways we probably want to add some namespace to the method names. In my own project I had a hell of a time debugging a broken POST body and found out the source to be the following category namespace collision. In NSURLRequest+postForm.m we add some category methods on to NSMutableData @@@ objectivec @interface NSMutableData (append) - (void)appendString:(NSString*)string; - (void)appendFormat:(NSString *)format, ...; @end @@@ Well it seems in 10.6 Apple has some similarly named methods that are colliding with ours. objc[205]: REPLACED: -[NSMutableData appendString:] by category MobileMeNSMutableDataAdditions (IMP was 0x8ac32 (/Users/zorn/Builds/Debug/ProfitTrain.app/Contents/MacOS/ProfitTrain), now 0x92ce23bd (/System/Library/PrivateFrameworks/DotMacSyncManager.framework/Versions/A/DotMacSyncManager)) I don't load the DotMacSyncManager framework and you won't see this collision with a generic new Cocoa app (like the demo app which works fine in 10.6). I assume it's coming along for the ride with WebKit or something I do link against. Anyways we probably want to add some namespace to the method names. <div><p>In my own project I had a hell of a time debugging a broken POST body and found out the source to be the following category namespace collision.</p> <p>In NSURLRequest+postForm.m we add some category methods on to NSMutableData</p> <pre> <code class="objectivec">@interface NSMutableData (append) - (void)appendString:(NSString*)string; - (void)appendFormat:(NSString *)format, ...; @end</code> </pre> <p>Well it seems in 10.6 Apple has some similarly named methods that are colliding with ours.</p> <p>objc[205]: REPLACED: -[NSMutableData appendString:] by category MobileMeNSMutableDataAdditions (IMP was 0x8ac32 (/Users/zorn/Builds/Debug/ProfitTrain.app/Contents/MacOS/ProfitTrain), now 0x92ce23bd (/System/Library/PrivateFrameworks/DotMacSyncManager.framework/Versions/A/DotMacSyncManager))</p> <p>I don't load the DotMacSyncManager framework and you won't see this collision with a generic new Cocoa app (like the demo app which works fine in 10.6). I assume it's coming along for the ride with WebKit or something I do link against.</p> <p>Anyways we probably want to add some namespace to the method names.</p></div> 0 true 2009-12-01T18:27:28-06:00 2456 11 scnetworkcheckreachabilitybyname-is-deprecated-in-106 180383 24800 resolved warning 2009-12-02T12:25:06-06:00 45213 rentzsch Justin Williams http://rentzsch.lighthouseapp.com/projects/24800/tickets/11 *Summary:* SCNetworkCheckReachabilityByName has been deprecated in 10.6. The following patch will fix it. Comments/tweaks welcomed. http://gist.github.com/246780 *Summary:* SCNetworkCheckReachabilityByName has been deprecated in 10.6. The following patch will fix it. Comments/tweaks welcomed. http://gist.github.com/246780 <div><p><em>Summary:</em> SCNetworkCheckReachabilityByName has been deprecated in 10.6. The following patch will fix it. Comments/tweaks welcomed.<br></p> <p><a href= "http://gist.github.com/246780">http://gist.github.com/246780</a></p></div> 0 true 2009-09-05T03:25:05-05:00 25228 10 rails-cant-parse-posted-dat 138169 24800 resolved bug multipart post rails 2009-09-08T16:54:28-05:00 45213 rentzsch mantas http://rentzsch.lighthouseapp.com/projects/24800/tickets/10 I want to use Ruby On Rails as a backend. However, every time when data is posted I get following error: Status: 500 Internal Server Error bad content body I want to use Ruby On Rails as a backend. However, every time when data is posted I get following error: Status: 500 Internal Server Error bad content body <div><p>I want to use Ruby On Rails as a backend.</p> <p>However, every time when data is posted I get following error:</p> <p>Status: 500 Internal Server Error<br> bad content body</p></div> 0 false 2009-08-25T00:29:16-05:00 67497 9 french-xib-compiles-with-a-warning-on-105 134591 24800 open 2009-08-27T19:52:32-05:00 45213 rentzsch ddribin http://rentzsch.lighthouseapp.com/projects/24800/tickets/9 Same problem as the English .xib had a while ago. Something about the segmented control. Same problem as the English .xib had a while ago. Something about the segmented control. <div><p>Same problem as the English .xib had a while ago. Something about the segmented control.</p></div> 0 true 2009-02-19T16:26:34-06:00 7462 5 using-showfeedbackwithbugdetails-results-in-all-message-text-being-bold 73441 24800 resolved 2009-08-06T14:00:36-05:00 45213 rentzsch Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/5 As I noted in the readme for 1.1.1 [NEW BUG] When using this new method the textView is given the bug report text via setString: this result in all the test being bold by default. The main reason I haven't fixed it to date, is I'm not very good with attributed text. As I noted in the readme for 1.1.1 [NEW BUG] When using this new method the textView is given the bug report text via setString: this result in all the test being bold by default. The main reason I haven't fixed it to date, is I'm not very good with attributed text. <div><p>As I noted in the readme for 1.1.1</p> <p>[NEW BUG] When using this new method the textView is given the bug report text via setString: this result in all the test being bold by default.</p> <p>The main reason I haven't fixed it to date, is I'm not very good with attributed text.</p></div> 45213 0 true 2009-07-17T07:49:19-05:00 63029 8 jrfeedbackprovider-demo-app-wont-complie 114371 24800 resolved localization 2009-08-06T14:00:00-05:00 45213 rentzsch ctshryock rentzsch http://rentzsch.lighthouseapp.com/projects/24800/tickets/8 I just pulled 88221115a3b511eb1011fc44e6e00ca777bf4e68 from last night and the demo app wont compile, it's looking for JRFeedbackProvider.xib in the lib folder but it's been moved to English.lproj (or French.lproj) in the move to add localization. I changed the path to the resource in the xcode project to fix this for now, was there a change to JRFeedbackProvider.xcodeproj that didn't make the commit? I just pulled 88221115a3b511eb1011fc44e6e00ca777bf4e68 from last night and the demo app wont compile, it's looking for JRFeedbackProvider.xib in the lib folder but it's been moved to English.lproj (or French.lproj) in the move to add localization. I changed the path to the resource in the xcode project to fix this for now, was there a change to JRFeedbackProvider.xcodeproj that didn't make the commit? <div><p>I just pulled 88221115a3b511eb1011fc44e6e00ca777bf4e68 from last night and the demo app wont compile, it's looking for JRFeedbackProvider.xib in the lib folder but it's been moved to English.lproj (or French.lproj) in the move to add localization.<br></p> <p>I changed the path to the resource in the xcode project to fix this for now, was there a change to JRFeedbackProvider.xcodeproj that didn't make the commit?</p></div> 0 true 2009-06-14T13:54:31-05:00 59896 7 nscfarray-objectatindex-index-1-or-possibly-larger-beyond-bounds-0 120894 24800 resolved 2009-07-16T00:12:05-05:00 45213 rentzsch Rainer http://rentzsch.lighthouseapp.com/projects/24800/tickets/7 I am getting the same error as in bug # 6, it shows in your sample app like this: *** -[NSCFArray objectAtIndex:]: index (-1( or possibly larger)) beyond bounds (0) I'm on 10.5.7, Xcode 3.1, 10.5|Release|i386. When this error occurs, you can click the button again, and everything is fine. But a third click produces the same error, the fourth click gets the desired window. I'm a Cocoa newbie, couldn't make the stack trace happen, but I am willing to try again, if you - or someone? - were to tell me how. By caveman-debugging I found that the error is triggered by this line in +showFeedbackWithBugDetails in JRFeedbackController.m: [gFeedbackController showWindow:self]; This line is roughly on line 57 of your code. I am getting the same error as in bug # 6, it shows in your sample app like this: *** -[NSCFArray objectAtIndex:]: index (-1( or possibly larger)) beyond bounds (0) I'm on 10.5.7, Xcode 3.1, 10.5|Release|i386. When this error occurs, you can click the button again, and everything is fine. But a third click produces the same error, the fourth click gets the desired window. I'm a Cocoa newbie, couldn't make the stack trace happen, but I am willing to try again, if you - or someone? - were to tell me how. By caveman-debugging I found that the error is triggered by this line in +showFeedbackWithBugDetails in JRFeedbackController.m: [gFeedbackController showWindow:self]; This line is roughly on line 57 of your code. <div><p>I am getting the same error as in bug # 6, it shows in your sample app like this: *** -[NSCFArray objectAtIndex:]: index (-1( or possibly larger)) beyond bounds (0) I'm on 10.5.7, Xcode 3.1, 10.5|Release|i386. When this error occurs, you can click the button again, and everything is fine. But a third click produces the same error, the fourth click gets the desired window. I'm a Cocoa newbie, couldn't make the stack trace happen, but I am willing to try again, if you - or someone? - were to tell me how. By caveman-debugging I found that the error is triggered by this line in +showFeedbackWithBugDetails in JRFeedbackController.m: [gFeedbackController showWindow:self]; This line is roughly on line 57 of your code.</p></div> 45213 0 true 2009-03-01T16:55:09-06:00 46600 6 bug-please-help 66757 24800 wontfix 2009-06-12T18:17:22-05:00 59896 Rainer sergmac (at gmail) rentzsch http://rentzsch.lighthouseapp.com/projects/24800/tickets/6 After i implement your solution in my project and press Provide Feedback button, i get next error in console: [33313:10b] *** -[NSCFArray objectAtIndex:]: index (-1( or possibly larger)) beyond bounds (0) BUT after i press second time on the button window loads and all works perfectly, please help me solve this problem. Tried the same just on your solution after downloading it without making any changes, same thing. Thanks a lot! Waiting for your answer. After i implement your solution in my project and press Provide Feedback button, i get next error in console: [33313:10b] *** -[NSCFArray objectAtIndex:]: index (-1( or possibly larger)) beyond bounds (0) BUT after i press second time on the button window loads and all works perfectly, please help me solve this problem. Tried the same just on your solution after downloading it without making any changes, same thing. Thanks a lot! Waiting for your answer. <div><p>After i implement your solution in my project and press Provide Feedback button, i get next error in console: [33313:10b] *** -[NSCFArray objectAtIndex:]: index (-1( or possibly larger)) beyond bounds (0) BUT after i press second time on the button window loads and all works perfectly, please help me solve this problem. Tried the same just on your solution after downloading it without making any changes, same thing. Thanks a lot! Waiting for your answer.</p></div> 0 true 2009-02-03T20:23:00-06:00 7462 1 bug-checkbox-for-email-is-not-always-honored 68064 24800 wontfix 2009-02-18T16:17:13-06:00 45213 rentzsch Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/1 When I bring forward the feedback window, deselect the checkbox to send my email and then click on the support tab, the checkbox is re-selected. I expect the checkbox to remain unselected. When I bring forward the feedback window, deselect the checkbox to send my email and then click on the support tab, the checkbox is re-selected. I expect the checkbox to remain unselected. <div><p>When I bring forward the feedback window, deselect the checkbox to send my email and then click on the support tab, the checkbox is re-selected.</p> <p>I expect the checkbox to remain unselected.</p></div> 0 true 2009-02-03T20:24:11-06:00 7462 2 feature-should-post-app-version-and-build-number-in-report 68065 24800 resolved 2009-02-18T16:16:20-06:00 45213 rentzsch Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/2 It would be very helpful to know what version of my app they are running when they submit feedback. If I notice they are running 1.2.2 I can say go download 1.2.4 to fix this issue. It would be very helpful to know what version of my app they are running when they submit feedback. If I notice they are running 1.2.2 I can say go download 1.2.4 to fix this issue. <div><p>It would be very helpful to know what version of my app they are running when they submit feedback. If I notice they are running 1.2.2 I can say go download 1.2.4 to fix this issue.</p></div> 0 true 2009-02-04T09:30:06-06:00 7462 4 feature-should-be-able-to-pre-fill-feedback-form-from-code 68225 24800 resolved 2009-02-18T16:16:09-06:00 45213 rentzsch Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/4 It would be nice if there was a way to init a feedback window with a pre-filled message. For example, in my persistence code I do all the sql inside a transaction, before i commit it I check to see if we has any errors during the transaction (bad syntax, whatever). I then rollback and explain to the user why we can't save. Next to the OK (this sucks) button I want to have a button "Report as Bug" which would open the feedback window with a pre-filled message, the errors I got from SQLite. It would be nice if there was a way to init a feedback window with a pre-filled message. For example, in my persistence code I do all the sql inside a transaction, before i commit it I check to see if we has any errors during the transaction (bad syntax, whatever). I then rollback and explain to the user why we can't save. Next to the OK (this sucks) button I want to have a button "Report as Bug" which would open the feedback window with a pre-filled message, the errors I got from SQLite. <div><p>It would be nice if there was a way to init a feedback window with a pre-filled message.</p> <p>For example, in my persistence code I do all the sql inside a transaction, before i commit it I check to see if we has any errors during the transaction (bad syntax, whatever). I then rollback and explain to the user why we can't save. Next to the OK (this sucks) button I want to have a button "Report as Bug" which would open the feedback window with a pre-filled message, the errors I got from SQLite.</p></div> 0 true 2009-02-03T20:25:38-06:00 7462 3 feature-users-should-consent-to-hardware-profile-info 68066 24800 resolved 2009-02-18T16:15:48-06:00 45213 rentzsch Mike Zornek http://rentzsch.lighthouseapp.com/projects/24800/tickets/3 I'd love to see a checkbox where the user acknowledges sending in their hardware profile. In my own setup these posts auto generate FogBugz tickets and users are sent a link where they can follow the new ticket. Imaging sending one line of feedback asking for a simple feature, and then viewing the ticket to discover I've collected pages of hardware details and who knows what else. That info is scary to users. I'd love to see a checkbox where the user acknowledges sending in their hardware profile. In my own setup these posts auto generate FogBugz tickets and users are sent a link where they can follow the new ticket. Imaging sending one line of feedback asking for a simple feature, and then viewing the ticket to discover I've collected pages of hardware details and who knows what else. That info is scary to users. <div><p>I'd love to see a checkbox where the user acknowledges sending in their hardware profile.</p> <p>In my own setup these posts auto generate FogBugz tickets and users are sent a link where they can follow the new ticket. Imaging sending one line of feedback asking for a simple feature, and then viewing the ticket to discover I've collected pages of hardware details and who knows what else. That info is scary to users.</p></div>