From mboxrd@z Thu Jan 1 00:00:00 1970 From: Allen Li Subject: Re: New feature? Remove duplicate subheadings, preserving order Date: Mon, 1 Jan 2018 23:40:12 -0800 Message-ID: References: <87a7xxx5c5.fsf@nicolasgoaziou.fr> <87tvw5cu5t.fsf@nicolasgoaziou.fr> <87mv1wvr70.fsf@alphapapa.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:40568) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eWHBJ-0008Pz-6O for emacs-orgmode@gnu.org; Tue, 02 Jan 2018 02:40:18 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eWHBG-00028h-Dn for emacs-orgmode@gnu.org; Tue, 02 Jan 2018 02:40:15 -0500 Received: from mail-qt0-x22e.google.com ([2607:f8b0:400d:c0d::22e]:38959) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eWHBG-00028E-5q for emacs-orgmode@gnu.org; Tue, 02 Jan 2018 02:40:14 -0500 Received: by mail-qt0-x22e.google.com with SMTP id k19so61791868qtj.6 for ; Mon, 01 Jan 2018 23:40:13 -0800 (PST) In-Reply-To: <87mv1wvr70.fsf@alphapapa.net> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: Adam Porter Cc: emacs-orgmode@gnu.org On Mon, Jan 1, 2018 at 8:07 PM, Adam Porter wrote: > Allen Li writes: > >> I don=E2=80=99t know if a more intelligent way of handling tags and todo >> keywords is worth the extra complexity, but in the use case that I >> imagine it makes sense to match using only the heading/list item: >> >> * Things to buy >> ** TODO cabbage >> ** DONE milk :store1: >> Maybe I forgot a tag here. Oh well, I already bought the milk. >> ** TODO carrots >> ** TODO milk :store1:store2: >> >> ... >> >> It doesn=E2=80=99t make sense to include the contents because I see this= as >> primarily being useful for list items. In particular, we would want >> to ignore log entries and properties for the sake of matching >> (intelligent property or logbook merging might be useful, but adds >> complexity). > > I think such a command should check all heading data by default, > because that's the safest option. A user who commonly needs to ignore > one or more types of data could use a custom function that calls the > command with arguments to disable checking of certain types. I don=E2=80=99t see a use case for checking all heading data. >> Since the point would be remove duplicates from lists, I don=E2=80=99t t= hink >> warning is very useful. I would want to remove the duplicate list >> items, not get a warning about it and delete them manually. Perhaps >> that would be a useful additional feature however (like uniq -d). > > I think warning or asking for confirmation should be the default action, > because it's the safest option. Users who want to skip that could use a > prefix argument or call it from a custom command. There is always undo and automatic Emacs file backups. >> I don=E2=80=99t think doing a full text check is useful, but if someone = has a >> use case for that, please speak up. > > An example where this would be useful is if the user has copied and > pasted subtrees and accidentally pasted one more than once. In that case, the user should use undo instead of a remove duplicates command. > I argue here for the safest behavior by default because I've found that, > in very large Org buffers, it's easy to lose my place in the file, and > it's easy to accidentally do something that I didn't mean to, without > noticing. IMO this is simply a consequence of Org buffers still being > plain-text. I don=E2=80=99t agree with this philosophy. Org and Emacs already has lots= of commands that can cause damage, for example org-sort-entries which my remove duplicates command is modeled after (both modify the direct children under the heading at point irreversibly ignoring undo). If this command sh= ould warn, then org-sort-entries should also warn. If org-sort-entries does not= need to warn, then this command does not need to warn. Emacs makes backups by default and supports undo, which under my philosophy is good enough; we shouldn=E2=80=99t be constantly asking for confirmation to prevent user error. That just causes pop-up dialog fatigue= . For example, everyone clicks OK on pop-up confirmation boxes without reading them. These kinds of confirmation prompts are worse than useless; they slow down and annoy the user without providing any protection. Undo is the better solution. > So it is quite conceivable to me that a user might intentionally give > two headings the same name (e.g. a user who captures quotations to an > inbox file might have two "Quote" headings that are completely > different), or might accidentally duplicate a subtree and then make a > desired change to one of them without realizing there was a > duplicate--then he might use this deduplication command and accidentally > delete a subtree he didn't mean to, resulting in data loss. I think it would be more useful for list members to post actual use cases than hypothesize about what people want. For me, the use case is filtering out duplicates from a list, e.g. groceries to buy or links to read captured with timestamps and other metadata, so checking the tags, todo, or body text is not useful, warning is not useful. Based on the responses I have gotten, it sounds like this feature is too specialized to be worth including in Org mode, so I will stop pursuing this unless people post actual use cases/desire for the feature.