Closed Bug 334521 Opened 19 years ago Closed 18 years ago

Initial shortcut/keyword not applied if URI changed when adding bookmark

Categories

(Firefox :: Bookmarks & History, defect)

x86
Windows Server 2003
defect
Not set
minor

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: ignazvonstraubing+bugzilla, Unassigned)

References

Details

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9a1) Gecko/20060416 Firefox/3.0a1 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9a1) Gecko/20060416 Firefox/3.0a1 When adding a new bookmark via Ctrl+D, if the URI is changed, the keyword/shortcut is ignored. The keyword can be added in after the bookmark has been added, however. Additionally, once the bookmark has been added the first time (showing empty shortcut field), pressing Ctrl+D on the same page will show the shortcut from the inital add. Reproducible: Always Steps to Reproduce: 1. Hit Ctrl+D to add new bookmark 2. Change Location to about:blank 3. Change shortcut field to 'blank' (no quotes) 4. Add Bookmark 5. Open bookmarks manager, check properties of new bookmark 6. Note that shortcut field is empty Actual Results: Shortcut field is empty on initial add Expected Results: Shortcut field should have specified term
*** Bug 335095 has been marked as a duplicate of this bug. ***
Probably we need another transaction for changeBookmarkURI as well... http://lxr.mozilla.org/mozilla/source/browser/components/places/content/bookmarkProperties.js > if (this._isVariant(this.EDIT_BOOKMARK_VARIANT) && > (newURI.spec != this._bookmarkURI.spec)) { > this._controller.changeBookmarkURI(this._bookmarkURI, newURI); > }
Status: UNCONFIRMED → NEW
Ever confirmed: true
*** Bug 354023 has been marked as a duplicate of this bug. ***
WFM on current places-bookmarks builds (wfm via the New Bookmark command, the keyword field isn't exposed for Accel+D anymore).
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.