Bug: org table updates [8.2.7 (8.2.7-elpaplus @ /Users/ctanis/.emacs.d/elpa/org-plus-contrib-20140613/)]

Bug: org table updates [8.2.7 (8.2.7-elpaplus @ /Users/ctanis/.emacs.d/elpa/org-plus-contrib-20140613/)]

[Craig Tanis]

Remember to cover the basics, that is, what you expected to happen and
what in fact did happen.  You don't know how to make a good report?  See

     http://orgmode.org/manual/Feedback.html#Feedback

Your bug report will be posted to the Org-mode mailing list.
------------------------------------------------------------------------

The code in org.el that checks for table movement to support
`org-table-auto-blank-field' has hardcoded an obsolete yasnippet
function.

Specifically, there's a line in `org-self-insert-command' that checks
`last-command' against a list of table movement functions to see if
auto-blank-field should occur.  This list includes 
`yas/expand' -- obsolesced in yasnippet 0.8, in favor of `yas-expand'.

In my opinion, this list should be a configurable variable, as other
minor modes may introduce functions that should also trigger auto-blank
behavior.  I also check for `autopair-newline' here, since this is the
binding for RET when autopair-mode is active.



Emacs  : GNU Emacs 24.3.1 (x86_64-apple-darwin13.1.0, NS apple-appkit-1265.19)
of 2014-05-05 on zartan.local
Package: Org-mode version 8.2.7 (8.2.7-elpaplus @ /Users/ctanis/.emacs.d/elpa/org-plus-contrib-20140613/)

current state:
==============
(setq
org-tab-first-hook '(org-hide-block-toggle-maybe org-src-native-tab-command-maybe
                      org-babel-hide-result-toggle-maybe org-babel-header-arg-expand)
org-goto-interface 'outline-path-completion
org-speed-command-hook '(org-speed-command-default-hook org-babel-speed-command-hook)
org-agenda-diary-file "~/Dropbox/org/diary.org"
org-mac-grab-Chrome-app-p nil
org-occur-hook '(org-first-headline-recenter)
org-metaup-hook '(org-babel-load-in-session-maybe)
org-log-done 'time
org-confirm-shell-link-function 'yes-or-no-p
org-image-actual-width nil
org-mac-grab-Firefox-app-p nil
org-startup-folded nil
org-yank-folded-subtrees nil
org-list-allow-alphabetical t
org-special-ctrl-a/e 'reversed
org-after-todo-state-change-hook '(org-clock-out-if-current)
org-src-mode-hook '(org-src-babel-configure-edit-buffer
                     org-src-mode-configure-edit-buffer)
org-agenda-before-write-hook '(org-agenda-add-entry-text)
org-babel-pre-tangle-hook '(save-buffer)
org-mode-hook '((lambda nil
                  (define-key org-mode-map (kbd "C-c g") (quote org-mac-grab-link)))
                 #[nil "\300\301\302\303\304$\207"
                   [org-add-hook change-major-mode-hook org-show-block-all append
                    local]
                   5]
                 #[nil "\300\301\302\303\304$\207"
                   [org-add-hook change-major-mode-hook org-babel-show-result-all
                    append local]
                   5]
                 org-babel-result-hide-spec org-babel-hide-all-hashes
                 (lambda nil (make-variable-buffer-local (quote yas/trigger-key))
                  (setq yas/trigger-key [tab])
                  (add-to-list (quote org-tab-first-hook)
                   (quote yas/org-very-safe-expand))
                  (define-key yas/keymap [tab] (quote yas/next-field)))
                 (lambda nil (local-set-key "\220" (quote org-backward-element))
                  (local-set-key "\216" (quote org-forward-element))
                  (local-set-key "\373" (quote backward-paragraph))
                  (local-set-key "\375" (quote forward-paragraph))
                  (local-set-key "\225" (quote org-up-list-or-element))
                  (local-set-key "\204" (quote org-down-element))
                  (local-set-key "\367" (quote org-refile-fullpath))
                  (local-set-key "\342" (quote org-mark-ring-goto))
                  (local-set-key "\356" (quote org-next-block))
                  (local-set-key "\360" (quote org-previous-block))
                  (auto-fill-mode 1) (toggle-truncate-lines -1)
                  (local-set-key (kbd "C-c SPC") (quote ace-jump-mode)))
                 )
org-refile-targets '((nil :maxlevel . 9))
org-outline-path-complete-in-steps nil
org-use-speed-commands t
org-ctrl-c-ctrl-c-hook '(org-babel-hash-at-point org-babel-execute-safely-maybe)
org-refile-use-outline-path t
org-directory "~/Dropbox/org"
org-cycle-hook '(org-cycle-hide-archived-subtrees org-cycle-hide-drawers
                  org-cycle-hide-inline-tasks org-cycle-show-empty-lines
                  org-optimize-window-after-visibility-change)
org-archive-default-command 'org-toggle-archive-tag
org-hide-emphasis-markers t
org-catch-invisible-edits 'smart
org-babel-tangle-lang-exts '(("python" . "py") ("java" . "java") ("C++" . "cpp")
                              ("perl" . "pl") ("emacs-lisp" . "el"))
org-confirm-elisp-link-function 'yes-or-no-p
org-metadown-hook '(org-babel-pop-to-session-maybe)
org-log-into-drawer t
org-completion-use-ido t
org-babel-load-languages '((emacs-lisp . t) (perl . t) (C . t) (java . t)
                            (python . t) (ditaa . t) (dot . t) (gnuplot . t)
                            (octave . t) (calc . t) (sh . t))
org-cycle-global-at-bob t
org-agenda-files '("~/Dropbox/org/notes.org"
                    "~/Dropbox/thesis/tex/chapters/outline.org"
                    "~/Dropbox/thesis/tex/chapters/implementation.org"
                    "~/Dropbox/thesis/tex/chapters/design.org"
                    "~/Dropbox/thesis/tex/chapters/conclusion.org"
                    "~/Dropbox/thesis/tex/chapters/background.org"
                    "~/Dropbox/thesis/tex/chapters/applications.org"
                    "~/Dropbox/research/splatt.org" "~/teaching/1110/1110-fa13.org"
                    "~/teaching/1100/1100-fa13.org")
org-clock-out-hook '(org-clock-remove-empty-clock-drawer)
)

Re: Bug: org table updates [8.2.7 (8.2.7-elpaplus @ /Users/ctanis/.emacs.d/elpa/org-plus-contrib-20140613/)]

[Bastien]

Hi Craig,

Craig Tanis <crt@weblar.org> writes:

> Specifically, there's a line in `org-self-insert-command' that checks
> `last-command' against a list of table movement functions to see if
> auto-blank-field should occur.  This list includes 
> `yas/expand' -- obsolesced in yasnippet 0.8, in favor of
> `yas-expand'.

It uses yas-expand now. 

> In my opinion, this list should be a configurable variable, as other
> minor modes may introduce functions that should also trigger auto-blank
> behavior.  I also check for `autopair-newline' here, since this is the
> binding for RET when autopair-mode is active.

I could not come up with a good name for an option, which should
probably start with "org-table-" -- if you have an idea, please let
me know.

Thanks,

-- 
 Bastien

Re: Bug: org table updates [8.2.7 (8.2.7-elpaplus @ /Users/ctanis/.emacs.d/elpa/org-plus-contrib-20140613/)]

[Craig Tanis]

[-- Attachment #1: Type: text/plain, Size: 932 bytes --]

Great news!
How about org-table-cell-blank-triggers ?
On Jul 28, 2014 1:25 PM, "Bastien" <bzg@gnu.org> wrote:

> Hi Craig,
>
> Craig Tanis <crt@weblar.org> writes:
>
> > Specifically, there's a line in `org-self-insert-command' that checks
> > `last-command' against a list of table movement functions to see if
> > auto-blank-field should occur.  This list includes
> > `yas/expand' -- obsolesced in yasnippet 0.8, in favor of
> > `yas-expand'.
>
> It uses yas-expand now.
>
> > In my opinion, this list should be a configurable variable, as other
> > minor modes may introduce functions that should also trigger auto-blank
> > behavior.  I also check for `autopair-newline' here, since this is the
> > binding for RET when autopair-mode is active.
>
> I could not come up with a good name for an option, which should
> probably start with "org-table-" -- if you have an idea, please let
> me know.
>
> Thanks,
>
> --
>  Bastien
>

[-- Attachment #2: Type: text/html, Size: 1361 bytes --]

Re: Bug: org table updates [8.2.7 (8.2.7-elpaplus @ /Users/ctanis/.emacs.d/elpa/org-plus-contrib-20140613/)]

[Bastien]

Hi Craig,

Craig Tanis <crt@weblar.org> writes:

> How about org-table-cell-blank-triggers ?

Sounds good.  Can you make a patch for this?

The variable needs to be in org-table.el and org.el needs
to use (defvar org-table-cell-blank-triggers) before it's
first use.

Thanks in advance,

-- 
 Bastien