Closed
Bug 52199
Opened 24 years ago
Closed 23 years ago
Unable to Paste duplicate URLs into same bookmark's folder
Categories
(SeaMonkey :: Bookmarks & History, defect, P3)
Tracking
(Not tracked)
People
(Reporter: jesup, Assigned: bugs)
Details
FreeBSD 4.1 20000910xx Tried to copy a few folders and then paste (in Manage Bookmarks) Got this JS error: ->>>>>>>>>>>>>> Read Clipboard from memory ************************************************************ ** NOTE: This report will only be printed in DEBUG builds.** * Call to xpconnect wrapped JSObject produced this error: * [Exception... "[JavaScript Warning: "assignment to undeclared variable pasteNodeRes" {file: "chrome://communicator/content/bookmarks/bookmarks.js" line: 339}]" nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)" location: "JS frame :: chrome://communicator/content/bookmarks/bookmarks.js :: doPaste :: line 339" data: yes] ************************************************************ An error occurred executing the cmd_paste command
Comment 1•24 years ago
|
||
rjesup, please list specific steps to repro with expected vs. actual results.
Reporter | ||
Comment 2•24 years ago
|
||
Open bookmark manager Select a bookmark Edit menu -> Copy Select another bookmark Edit menu -> Paste Expected result: bookmark is pasted Actual result: error as given
Comment 3•24 years ago
|
||
Hmmm, I can't be of much help here. Copy and paste works fine on Linux RH6 with the 2000091308 build. The caveat being that you of course can't copy a bookmark to the same level of the tree hierarchy that it already exists on. So to test I naturally pasted my copy into a folder. Trying to paste at the same level failed silently.
Reporter | ||
Comment 4•24 years ago
|
||
I _was_ trying to paste it at the same level. There was no reason I should assume it shouldn't work (I can see the logic of that, but it's not intuitive to the user! At least not without notification!) You didn't get the js exception below? Were you using a --enable-debug build? Sounds like the error in pasting (duplicate) is being return to the JS which doesn't handle the error.
Comment 5•24 years ago
|
||
I should note I'm using optimized builds. I do get js errors in my terminal window but not assertions and whatnot.
Reporter | ||
Comment 6•24 years ago
|
||
Aha. As per the output: ** NOTE: This report will only be printed in DEBUG builds.** So, I'd say there are two things here: one that it silently doesn't paste a link at the same level (which is at least confusing), and two that it when it fails to paste a link it gets an unhandled error (exception) (which if it pasted the link would not be an issue). Is the disallowing of duplicate links in the same folder a formal UI decision? Is it an artifact of the code, or an off-the-cuff decision by the implementer? I know ns4.7 and I believe IE allow duplicate links at the same level.
Comment 7•24 years ago
|
||
seeing as it's slamm's last day i'm reassigning to, instead of cc'ing rjc. Robert, there are few bugs floating around that touch on the issue of duplicate bookmarks, and the ramications of having/trying to create them, care to comment?
Assignee: slamm → rjc
Updated•24 years ago
|
Summary: Paste doesn't work in bookmarks → Unable to Paste duplicate URLs into same bookmark's folder
Target Milestone: --- → Future
Comment 8•24 years ago
|
||
Netscape Nav Triage Team: dupe, depends on single bookmarks.
Comment 9•24 years ago
|
||
Randell, is this still happening in the new bookmarks manager?
Reporter | ||
Comment 10•24 years ago
|
||
Still there. The only thing that changed is that it doesn't print an error. To repeat my last comment: Is the disallowing of duplicate links in the same folder a formal UI decision? Is it an artifact of the code, or an off-the-cuff decision by the implementer? I know ns4.7 and I believe IE allow duplicate links at the same level. I strongly object to this behavior, at least without notification. It breaks the user's expectations about how copy and paste work. If we wanted to add a requester saying "You already have a bookmark to that site in this folder, do you want to add another copy?", that'd be ok. Removing target 'future', adding nomination for 1.0
Keywords: mozilla1.0
Target Milestone: Future → ---
Comment 11•24 years ago
|
||
sounds good and fair to me
Comment 12•23 years ago
|
||
to answer randell, as I understand it this is an artifact of the code. I believe it has since been reworked to not have that dependency anymore. I believe this bug has XP implications and i believe it has most likely been dealt with elsewhere whilst this bug has been languishing assinged to someone no longer affiliated with the project.
Assignee: rjc → ben
Reporter | ||
Comment 13•23 years ago
|
||
This is still a bug as of about a week ago, and I just retested with a build from yesterday and it's still there, so I don't think it's been fixed in the meantime (unless you mean that there's something it depends on that was broken and has since been fixed, and no one has updated bookmarks yet).
Comment 14•23 years ago
|
||
ok. I think this is a dupe, it's just a question of finding it...
Whiteboard: dupeme
Comment 15•23 years ago
|
||
I believe this is a dupe of bug 41502. That's just a more general case where the folder in question is the root bookmarks folder.
Whiteboard: dupeme
Reporter | ||
Comment 16•23 years ago
|
||
Agreed, DUP *** This bug has been marked as a duplicate of 41502 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•