On Mon, Oct 25, 2004 at 04:13:23PM +0300, Martin Vermeer wrote:
> 
> At least this looks correct now... but can't you (referring to before
> this proposed patch) just place the counters thingy *inside* the second
> if-statement? Again, what do I miss?
> 
> Like this
> 
>         if (!bstyle->latexparam().empty()) {
>                 id = bstyle->latexparam();
>                 if (id.find('#') != string::npos) {
>                         counters.step(bstyle->counter);
>                         string el = expandLabel(buf.params().getLyXTextClass(),
>                                                 bstyle, false);
>                         id = subst(id, "#", el);
>                 }
>         }

  With this code you are overriding the id even if you don't have any # in
your style parameter.

  Then probably the first line inside the if should be instead:
  
  id += " " + bstyle->latexparam();

> -- 
> Martin Vermeer <[EMAIL PROTECTED]>

-- 
José Abílio Matos
LyX and docbook a perfect match. :-)

Reply via email to