Hi again

Rationale of the patch: 
Currently, when the options to biblatex are _not_ xxx/xxx format, a style= is prepended, because
it is assumed that it will be a biblatex options string starting with a style. 
With the proposed patch, I add the possibility of using a biblatex options string where style= is stated
explicitly (anywhere in the string). This adds congruency WRT the ~org-cite-biblatex-options~ variable.

Best, /PA

On Sun, 14 Apr 2024 at 09:16, Pedro Andres Aranda Gutierrez <paaguti@gmail.com> wrote:
HI, full answer... 

In the original code, that section prepends "style=" in some cases. I have stumbled across
instances where the options started with style= because they were taken directly from LaTeX
and the generated code ended by like:
\usepackage[style=style=numeric,backend=biber]{biblatex}
for 
#+cite_export: biblatex style=numeric,backend=biber
which, BTW, is something someone used the LaTeX may parse as correct.
I was just trying to cater for this case.

Best, /PA


On Sun, 14 Apr 2024 at 07:30, Pedro Andres Aranda Gutierrez <paaguti@gmail.com> wrote:
Sorry... I was not right... You are right. I'll take a look at it again.
/PA

On Sun, 14 Apr 2024 at 07:22, Pedro Andres Aranda Gutierrez <paaguti@gmail.com> wrote:
Hi,

Maybe not... because style= is a keyword for the biblatex package, while bibstyle or citestyle aren't. 

Best, /PA

On Sat, 13 Apr 2024 at 16:32, Ihor Radchenko <yantar92@posteo.net> wrote:
Pedro Andres Aranda Gutierrez <paaguti@gmail.com> writes:

> HI,
> Attached is a small patch to allow explicitly adding style= in the biblatex export options, to increase consistency with
> Customisation variables.

Thanks!

>          (style-options
>           (cond
>            ((null style) nil)
> +          ;; allow the user to include "style=" anywhere in the style options
> +          ((string-match "\\(^s\\|,s\\)tyle=" style) (list style))
>            ((not (string-match "/" style)) (list (concat "style=" style)))
>            (t
>             (list (concat "bibstyle=" (substring style nil (match-beginning 0)))

If we allow style=..., may as well allow bibstyle= and citestyle=.

--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>


--
Fragen sind nicht da, um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler

Headaches with a Juju log:
unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run a leader-deposed hook here, but we can't yet



--
Fragen sind nicht da, um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler

Headaches with a Juju log:
unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run a leader-deposed hook here, but we can't yet



--
Fragen sind nicht da, um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler

Headaches with a Juju log:
unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run a leader-deposed hook here, but we can't yet



--
Fragen sind nicht da, um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler

Headaches with a Juju log:
unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run a leader-deposed hook here, but we can't yet