Bug 94212 - can't navigate back one page when result is 404 not found
Summary: can't navigate back one page when result is 404 not found
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
: 94527 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-12-01 12:36 UTC by Luther
Modified: 2007-12-11 20:17 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Luther 2004-12-01 12:36:50 UTC
Version:           1.0-beta7 "Lippel" (using KDE 3.3.1,  (3.1))
Compiler:          gcc version 3.3.5 (Debian 1:3.3.5-2)
OS:                Linux (i686) release 2.4.26

If the RSS feed article links to a standard web page and that page links you to a '404 Not found' error there is no way to go back one page to the last good page.

The user has to go back out to the originating feed and start back in from the originating article.  The user also has to close the tab where the 404 occurred.
Comment 1 Sashmit Bhaduri 2004-12-01 15:13:39 UTC
Have you tried beta8?
Comment 2 Luther 2004-12-01 20:54:31 UTC
Sashmit Bhaduri wrote:

>------- You are receiving this mail because: -------
>You reported the bug, or are watching the reporter.
>        
>http://bugs.kde.org/show_bug.cgi?id=94212        
>
>
>
>
>------- Additional Comments From smt vfemail net  2004-12-01 15:13 -------
>Have you tried beta8?
>
>  
>
Not yet.  Debian tends to runs a slightly behind.  Beta 7 is what is in 
the latest available so far in Deb Unstable.

That is the reason I only marked it as a wishlist just in case it was 
already fixed.  

If in Beta8 you can't get caught in a hyperlinked dead end in, then I 
will be good when it makes it into Debian.

Thank you for the suggestion.

Luther

=====

A way to replicate (but not the only way):

    * Add this feed:  http://rss.gmane.org/gmane.linux.debian.devel.general
    * pick any article and click "complete story"
    * left panel of GMANE, click HOME, then click RSS Feeds link (right
      window)
    * click on the link http://rss.gmane.org/gmane.linux.kernel

Basically, anywhere right in here you don't have a way back out because 
there are no links forward and no links backward.  Same holds true if 
you end up at a 404 not found page.


<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
  <title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Sashmit Bhaduri wrote:
<blockquote cite="mid20041201141342.22858.qmail@ktown.kde.org"
 type="cite">
  <pre wrap="">------- You are receiving this mail because: -------
You reported the bug, or are watching the reporter.
        
<a class="moz-txt-link-freetext" href="http://bugs.kde.org/show_bug.cgi?id=94212">http://bugs.kde.org/show_bug.cgi?id=94212</a>        




------- Additional Comments From smt vfemail net  2004-12-01 15:13 -------
Have you tried beta8?

  </pre>
</blockquote>
Not yet.  Debian tends to runs a slightly behind.  Beta 7 is what is in
the latest available so far in Deb Unstable.<br>
<br>
That is the reason I only marked it as a wishlist just in case it was
already fixed.   <br>
<br>
If in Beta8 you can't get caught in a hyperlinked dead end in, then I
will be good when it makes it into Debian. <br>
<br>
Thank you for the suggestion.<br>
<br>
Luther<br>
<br>
=====<br>
<br>
A way to replicate (but not the only way):<br>
<ul>
  <li>Add this feed: 
<a class="moz-txt-link-freetext" href="http://rss.gmane.org/gmane.linux.debian.devel.general">http://rss.gmane.org/gmane.linux.debian.devel.general</a></li>
  <li>pick any article and click "complete story"</li>
  <li>left panel of GMANE, click HOME, then click RSS Feeds link (right
window)</li>
  <li>click on the link <a class="moz-txt-link-freetext" href="http://rss.gmane.org/gmane.linux.kernel">http://rss.gmane.org/gmane.linux.kernel</a></li>
</ul>
Basically, anywhere right in here you don't have a way back out because
there are no links forward and no links backward.  Same holds true if
you end up at a 404 not found page. <br>
<br>
<br>
</body>
</html>
Comment 3 Sashmit Bhaduri 2004-12-02 17:21:16 UTC
I know pierre already packaged beta8.. I guess it just needs to enter unstable. 
Comment 4 George Staikos 2004-12-04 05:08:18 UTC
Verified, works fine in HEAD.
Comment 5 George Staikos 2004-12-06 09:10:47 UTC
Aha, the problem reproducing this is that you have to remove the focus from the khtml part.  Then the actions revert to the akregator part actions instead of the KHTML part actions.  Still valid.
Comment 6 George Staikos 2004-12-06 09:11:06 UTC
*** Bug 94527 has been marked as a duplicate of this bug. ***
Comment 7 George Staikos 2004-12-06 11:23:55 UTC
CVS commit by staikos: 

don't switch kpart's incorrectly
BUG: 94212


  M +6 -0      akregator_part.cpp   1.111
  M +2 -1      akregator_part.h   1.45
  M +2 -0      akregator_view.h   1.73


--- kdenonbeta/akregator/src/akregator_part.cpp  #1.110:1.111
@@ -35,4 +35,5 @@
 #include "akregator.h"
 #include "fetchtransaction.h"
+#include "frame.h"
 #include "myarticle.h"
 #include "settings_archive.h"
@@ -721,4 +722,9 @@ void aKregatorPart::showOptions()
 }
 
+KParts::Part *aKregatorPart::hitTest(QWidget *widget, const QPoint &globalPos)
+{
+    return m_view->currentFrame()->part();
+}
+
 #include "akregator_part.moc"
 

--- kdenonbeta/akregator/src/akregator_part.h  #1.44:1.45
@@ -170,4 +170,5 @@ namespace Akregator
             QWidget* getMainWindow();
 
+            virtual KParts::Part *hitTest(QWidget *widget, const QPoint &globalPos);
             
         public slots:

--- kdenonbeta/akregator/src/akregator_view.h  #1.72:1.73
@@ -115,4 +115,6 @@ namespace Akregator
             virtual void saveProperties(KConfig* config);
             
+            Frame *currentFrame() const { return m_currentFrame; }
+            
         public slots:
             


Comment 8 Luther 2004-12-12 15:55:49 UTC
Confirmed fixed in Beta8 for Debian by originator of this bug.

Thank you.  Luther