org-lms icon indicating copy to clipboard operation
org-lms copied to clipboard

Use Dash & let-alist to greatly simplify code

Open titaniumbones opened this issue 6 years ago • 1 comments

Code in these modules is extremely repetitive and somewhat brittle. Much of this brittle repetition involves reading and setting headline properties in org-mode. Adam Porter & John Kitchin have helpfully suggested using let-alist and dash's &plist syntax to easily set/rename many variables at a time, using org-entry-properties to first gather them all up.

org-process-props in recent commits is a good start. Actually refactoring all the ocde feels a bit challenging at present but would be a really good idea.

titaniumbones avatar Sep 30 '19 14:09 titaniumbones

still stalledo n this, but converting to map.el helps a little. The convusion around plists and alists is awful here.

titaniumbones avatar Apr 30 '21 13:04 titaniumbones