]> git.lyx.org Git - features.git/blobdiff - src/output_docbook.cpp
Move include of own header to the top. Fix dependencies
[features.git] / src / output_docbook.cpp
index 3e37f9091265a0918025e1b99b170f34b1a34cf4..63cae8a58bb69ae9b42ba0bf604cb8a4fa3110e1 100644 (file)
@@ -11,6 +11,8 @@
 
 #include <config.h>
 
+#include "output_docbook.h"
+
 #include "Buffer.h"
 #include "buffer_funcs.h"
 #include "BufferParams.h"
 #include "insets/InsetBibtex.h"
 #include "insets/InsetBibitem.h"
 #include "insets/InsetLabel.h"
+#include "mathed/InsetMath.h"
 #include "insets/InsetNote.h"
 
 #include "support/lassert.h"
+#include "support/textutils.h"
 
 #include <stack>
 #include <iostream>
@@ -49,7 +53,7 @@ std::string fontToDocBookTag(xml::FontTypes type)
        case xml::FontTypes::FT_BOLD:
                return "emphasis";
        case xml::FontTypes::FT_NOUN:
-               return "person";
+               return "personname";
        case xml::FontTypes::FT_UBAR:
        case xml::FontTypes::FT_WAVE:
        case xml::FontTypes::FT_DBAR:
@@ -255,6 +259,25 @@ void closeTag(XMLStream & xs, const std::string & tag, const std::string & tagty
 }
 
 
+void compTag(XMLStream & xs, const std::string & tag, const std::string & attr, const std::string & tagtype)
+{
+       if (tag.empty() || tag == "NONE")
+               return;
+
+       // Special case for <para>: always considered as a paragraph.
+       if (tag == "para" || tagtype == "paragraph" || tagtype == "block") {
+               if (!xs.isLastTagCR())
+                       xs << xml::CR();
+               xs << xml::CompTag(tag, attr);
+               xs << xml::CR();
+       } else if (tagtype == "inline") {
+               xs << xml::CompTag(tag, attr);
+       } else {
+               xs.writeError("Unrecognised tag type '" + tagtype + "' for '" + tag + "'");
+       }
+}
+
+
 // Higher-level convenience functions.
 
 void openParTag(XMLStream & xs, const Paragraph * par, const Paragraph * prevpar)
@@ -274,8 +297,11 @@ void openParTag(XMLStream & xs, const Paragraph * par, const Paragraph * prevpar
        if (prevpar != nullptr) {
                Layout const & prevlay = prevpar->layout();
                if (prevlay.docbookwrappertag() != "NONE") {
-                       openWrapper = prevlay.docbookwrappertag() == lay.docbookwrappertag()
-                                     && !lay.docbookwrappermergewithprevious();
+                       if (prevlay.docbookwrappertag() == lay.docbookwrappertag() &&
+                                       prevlay.docbookwrapperattr() == lay.docbookwrapperattr())
+                               openWrapper = !lay.docbookwrappermergewithprevious();
+                       else
+                               openWrapper = true;
                }
        }
 
@@ -286,10 +312,12 @@ void openParTag(XMLStream & xs, const Paragraph * par, const Paragraph * prevpar
        const string & tag = lay.docbooktag();
        if (tag != "NONE") {
                auto xmltag = xml::ParTag(tag, lay.docbookattr());
-               if (!xs.isTagOpen(xmltag, 1)) // Don't nest a paragraph directly in a paragraph.
+               if (!xs.isTagOpen(xmltag, 1)) // Don't nest a paragraph directly in a paragraph.
                        // TODO: required or not?
                        // TODO: avoid creating a ParTag object just for this query...
                        openTag(xs, lay.docbooktag(), lay.docbookattr(), lay.docbooktagtype());
+                       openTag(xs, lay.docbookinnertag(), lay.docbookinnerattr(), lay.docbookinnertagtype());
+               }
        }
 
        openTag(xs, lay.docbookitemtag(), lay.docbookitemattr(), lay.docbookitemtagtype());
@@ -308,51 +336,24 @@ void closeParTag(XMLStream & xs, Paragraph const * par, Paragraph const * nextpa
        if (nextpar != nullptr) {
                Layout const & nextlay = nextpar->layout();
                if (nextlay.docbookwrappertag() != "NONE") {
-                       closeWrapper = nextlay.docbookwrappertag() == lay.docbookwrappertag()
-                                      && !nextlay.docbookwrappermergewithprevious();
+                       if (nextlay.docbookwrappertag() == lay.docbookwrappertag() &&
+                                       nextlay.docbookwrapperattr() == lay.docbookwrapperattr())
+                               closeWrapper = !nextlay.docbookwrappermergewithprevious();
+                       else
+                               closeWrapper = true;
                }
        }
 
        // Main logic.
        closeTag(xs, lay.docbookiteminnertag(), lay.docbookiteminnertagtype());
        closeTag(xs, lay.docbookitemtag(), lay.docbookitemtagtype());
+       closeTag(xs, lay.docbookinnertag(), lay.docbookinnertagtype());
        closeTag(xs, lay.docbooktag(), lay.docbooktagtype());
        if (closeWrapper)
                closeTag(xs, lay.docbookwrappertag(), lay.docbookwrappertagtype());
 }
 
 
-void openLabelTag(XMLStream & xs, Layout const & lay) // Mostly for definition lists.
-{
-       openTag(xs, lay.docbookitemlabeltag(), lay.docbookitemlabelattr(), lay.docbookitemlabeltagtype());
-}
-
-
-void closeLabelTag(XMLStream & xs, Layout const & lay)
-{
-       closeTag(xs, lay.docbookitemlabeltag(), lay.docbookitemlabeltagtype());
-}
-
-
-void openItemTag(XMLStream & xs, Layout const & lay)
-{
-       openTag(xs, lay.docbookitemtag(), lay.docbookitemattr(), lay.docbookitemtagtype());
-}
-
-
-void closeItemTag(XMLStream & xs, Layout const & lay)
-{
-       closeTag(xs, lay.docbookitemtag(), lay.docbookitemtagtype());
-}
-
-
-ParagraphList::const_iterator makeAny(Text const &,
-                                             Buffer const &,
-                                             XMLStream &,
-                                             OutputParams const &,
-                                             ParagraphList::const_iterator);
-
-
 void makeBibliography(
                Text const & text,
                Buffer const & buf,
@@ -361,8 +362,8 @@ void makeBibliography(
                ParagraphList::const_iterator const & par)
 {
        // If this is the first paragraph in a bibliography, open the bibliography tag.
-       auto pbegin_before = text.paragraphs().getParagraphBefore(par);
-       if (pbegin_before->layout().latextype != LATEX_BIB_ENVIRONMENT) {
+       auto const * pbegin_before = text.paragraphs().getParagraphBefore(par);
+       if (pbegin_before == nullptr || (pbegin_before && pbegin_before->layout().latextype != LATEX_BIB_ENVIRONMENT)) {
                xs << xml::StartTag("bibliography");
                xs << xml::CR();
        }
@@ -393,12 +394,9 @@ void makeBibliography(
 
        // If this is the last paragraph in a bibliography, close the bibliography tag.
        auto const end = text.paragraphs().end();
-       bool endBibliography = par == end;
-       if (!endBibliography) {
-               auto nextpar = par;
-               ++nextpar;
-               endBibliography = par->layout().latextype != LATEX_BIB_ENVIRONMENT;
-       }
+       auto nextpar = par;
+       ++nextpar;
+       bool endBibliography = nextpar == end || nextpar->layout().latextype != LATEX_BIB_ENVIRONMENT;
 
        if (endBibliography) {
                xs << xml::EndTag("bibliography");
@@ -451,29 +449,35 @@ void makeParagraph(
        // Plain layouts must be ignored.
        special_case |= buf.params().documentClass().isPlainLayout(par->layout()) && !runparams.docbook_force_pars;
        // Equations do not deserve their own paragraph (DocBook allows them outside paragraphs).
-       special_case |= nInsets == par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
-               return inset.inset && inset.inset->asInsetMath();
+       // Exception: any case that generates an <inlineequation> must still get a paragraph to be valid.
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset && inset.inset->asInsetMath() && inset.inset->asInsetMath()->getType() != hullSimple;
+       });
+       // Floats cannot be in paragraphs.
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset->lyxCode() == FLOAT_CODE;
+       });
+       // Bibliographies cannot be in paragraphs. Bibitems should still be handled as paragraphs, though
+       // (see makeParagraphBibliography).
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset->lyxCode() == BIBTEX_CODE;
+       });
+       // ERTs are in comments, not paragraphs.
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset->lyxCode() == ERT_CODE;
+       });
+       // Listings should not get into their own paragraph.
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset->lyxCode() == LISTINGS_CODE;
+       });
+       // Boxes cannot get into their own paragraph.
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset->lyxCode() == BOX_CODE;
+       });
+       // Includes should not have a paragraph.
+       special_case |= nInsets == (size_t) par->size() && std::all_of(par->insetList().begin(), par->insetList().end(), [](InsetList::Element inset) {
+               return inset.inset->lyxCode() == INCLUDE_CODE;
        });
-
-       // TODO: Could get rid of this with a DocBook equivalent to htmlisblock? Not for all cases, unfortunately... See above for those that have been determined not to be allowable for this potential refactoring.
-       if (!special_case && par->size() == 1 && par->getInset(0)) {
-               Inset const * firstInset = par->getInset(0);
-
-               // Floats cannot be in paragraphs.
-               special_case = to_utf8(firstInset->layoutName()).substr(0, 6) == "Float:";
-
-               // Bibliographies cannot be in paragraphs.
-               if (!special_case && firstInset->asInsetCommand())
-                       special_case = firstInset->asInsetCommand()->params().getCmdName() == "bibtex";
-
-               // ERTs are in comments, not paragraphs.
-               if (!special_case && firstInset->lyxCode() == lyx::ERT_CODE)
-                       special_case = true;
-
-               // Listings should not get into their own paragraph.
-               if (!special_case && firstInset->lyxCode() == lyx::LISTINGS_CODE)
-                       special_case = true;
-       }
 
        bool const open_par = runparams.docbook_make_pars
                                                  && !runparams.docbook_in_par
@@ -484,8 +488,6 @@ void makeParagraph(
        //              or we're not in the last paragraph, anyway.
        //   (ii) We didn't open it and docbook_in_par is true,
        //              but we are in the first par, and there is a next par.
-       auto nextpar = par;
-       ++nextpar;
        bool const close_par = open_par && (!runparams.docbook_in_par);
 
        // Determine if this paragraph has some real content. Things like new pages are not caught
@@ -493,9 +495,11 @@ void makeParagraph(
        // Thus, remove all spaces (including new lines: \r, \n) before checking for emptiness.
        // std::all_of allows doing this check without having to copy the string.
        // Open and close tags around each contained paragraph.
-       auto pars = par->simpleDocBookOnePar(buf, runparams, text.outerFont(distance(begin, par)), 0);
-       for (auto & parXML : pars) {
-               if (!std::all_of(parXML.begin(), parXML.end(), ::isspace)) {
+       auto nextpar = par;
+       ++nextpar;
+       auto pars = par->simpleDocBookOnePar(buf, runparams, text.outerFont(distance(begin, par)), 0, nextpar == end, special_case);
+       for (docstring const & parXML : pars) {
+               if (xml::isNotOnlySpace(parXML)) {
                        if (open_par)
                                openParTag(xs, &*par, prevpar);
 
@@ -514,8 +518,19 @@ void makeEnvironment(Text const &text,
                      OutputParams const &runparams,
                      ParagraphList::const_iterator const & par)
 {
-       // TODO: simplify me!
        auto const end = text.paragraphs().end();
+       auto nextpar = par;
+       ++nextpar;
+
+       // Special cases for listing-like environments provided in layouts. This is quite ad-hoc, but provides a useful
+       // default. This should not be used by too many environments (only LyX-Code right now).
+       // This would be much simpler if LyX-Code was implemented as InsetListings...
+       bool mimicListing = false;
+       bool ignoreFonts = false;
+       if (par->layout().docbooktag() == "programlisting") {
+               mimicListing = true;
+               ignoreFonts = true;
+       }
 
        // Output the opening tag for this environment, but only if it has not been previously opened (condition
        // implemented in openParTag).
@@ -527,65 +542,25 @@ void makeEnvironment(Text const &text,
        if (style.latextype == LATEX_COMMAND) {
                // Nothing to do (otherwise, infinite loops).
        } else if (style.latextype == LATEX_ENVIRONMENT) {
-               // Open a wrapper tag if needed.
-               if (style.docbookitemwrappertag() != "NONE")
-                       openTag(xs, style.docbookitemwrappertag(), style.docbookitemwrapperattr(), style.docbookitemwrappertagtype());
+               // Generate the paragraph, if need be.
+               auto pars = par->simpleDocBookOnePar(buf, runparams, text.outerFont(std::distance(text.paragraphs().begin(), par)), 0, false, ignoreFonts);
 
-               // Generate the label, if need be. If it is taken from the text, sep != 0 and corresponds to the first
-               // character after the label.
-               pos_type sep = 0;
-               if (style.labeltype != LABEL_NO_LABEL && style.docbookitemlabeltag() != "NONE") {
-                       // At least one condition must be met:
-                       //  - this environment is not a list
-                       //  - if this is a list, the label must not be manual (i.e. it must be taken from the layout)
-                       if (style.latextype != LATEX_LIST_ENVIRONMENT || style.labeltype != LABEL_MANUAL) {
-                               // Usual cases: maybe there is something specified at the layout level. Highly unlikely, though.
-                               docstring const lbl = par->params().labelString();
-
-                               if (!lbl.empty()) {
-                                       openLabelTag(xs, style);
-                                       xs << lbl;
-                                       closeLabelTag(xs, style);
-                               }
-                       } else {
-                               // Only variablelist gets here (or similar items defined as an extension in the layout).
-                               openLabelTag(xs, style);
-                               sep = par->firstWordDocBook(xs, runparams);
-                               closeLabelTag(xs, style);
-                       }
-               }
-
-               // Maybe the item is completely empty, i.e. if the first word ends at the end of the current paragraph
-               // AND if the next paragraph doesn't have the same depth (if there is such a paragraph).
-               // Common case: there is only the first word on the line, but there is a nested list instead
-               // of more text.
-               bool emptyItem = false;
-               if (sep == par->size()) { // If the separator is already at the end of this paragraph...
-                       auto next_par = par;
-                       ++next_par;
-                       if (next_par == text.paragraphs().end()) // There is no next paragraph.
-                               emptyItem = true;
-                       else // There is a next paragraph: check depth.
-                               emptyItem = par->params().depth() >= next_par->params().depth();
-               }
-
-               if (emptyItem) {
-                       // Avoid having an empty item, this is not valid DocBook. A single character is enough to force
-                       // generation of a full <para>.
-                       // TODO: this always worked only by magic...
-                       xs << ' ';
-               } else {
-                       // Generate the rest of the paragraph, if need be. Open as many inner tags as necessary.
-                       auto pars = par->simpleDocBookOnePar(buf, runparams, text.outerFont(std::distance(text.paragraphs().begin(), par)), sep);
+               if (mimicListing) {
                        auto p = pars.begin();
-                       while (true) {
+                       while (p != pars.end()) {
+                               openTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnerattr(), par->layout().docbookiteminnertagtype());
                                xs << XMLStream::ESCAPE_NONE << *p;
+                               closeTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnertagtype());
                                ++p;
-                               if (p != pars.end()) {
-                                       closeTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnertagtype());
-                                       openTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnerattr(), par->layout().docbookiteminnertagtype());
-                               } else
-                                       break;
+
+                               if (p != pars.end())
+                                       xs << xml::CR();
+                       }
+               } else {
+                       for (auto const & p : pars) {
+                               openTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnerattr(), par->layout().docbookiteminnertagtype());
+                               xs << XMLStream::ESCAPE_NONE << p;
+                               closeTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnertagtype());
                        }
                }
        } else {
@@ -593,8 +568,6 @@ void makeEnvironment(Text const &text,
        }
 
        // Close the environment.
-       auto nextpar = par;
-       ++nextpar;
        closeParTag(xs, &*par, (nextpar != end) ? &*nextpar : nullptr); // TODO: switch in layout for par/block?
 }
 
@@ -661,17 +634,17 @@ ParagraphList::const_iterator makeListEnvironment(Text const &text,
                if (style.labeltype != LABEL_NO_LABEL && style.docbookitemlabeltag() != "NONE") {
                        if (style.labeltype == LABEL_MANUAL) {
                                // Only variablelist gets here (or similar items defined as an extension in the layout).
-                               openLabelTag(xs, style);
+                               openTag(xs, style.docbookitemlabeltag(), style.docbookitemlabelattr(), style.docbookitemlabeltagtype());
                                sep = 1 + par->firstWordDocBook(xs, runparams);
-                               closeLabelTag(xs, style);
+                               closeTag(xs, style.docbookitemlabeltag(), style.docbookitemlabeltagtype());
                        } else {
                                // Usual cases: maybe there is something specified at the layout level. Highly unlikely, though.
                                docstring const lbl = par->params().labelString();
 
                                if (!lbl.empty()) {
-                                       openLabelTag(xs, style);
+                                       openTag(xs, style.docbookitemlabeltag(), style.docbookitemlabelattr(), style.docbookitemlabeltagtype());
                                        xs << lbl;
-                                       closeLabelTag(xs, style);
+                                       closeTag(xs, style.docbookitemlabeltag(), style.docbookitemlabeltagtype());
                                }
                        }
                }
@@ -691,9 +664,8 @@ ParagraphList::const_iterator makeListEnvironment(Text const &text,
                        }
                } else {
                        // DocBook doesn't like emptiness.
-                       openTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnerattr(),
+                       compTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnerattr(),
                                par->layout().docbookiteminnertagtype());
-                       closeTag(xs, par->layout().docbookiteminnertag(), par->layout().docbookiteminnertagtype());
                }
 
                // If the next item is deeper, it must go entirely within this item (do it recursively).
@@ -744,35 +716,6 @@ void makeCommand(
 }
 
 
-ParagraphList::const_iterator makeAny(Text const &text,
-                                             Buffer const &buf,
-                                             XMLStream &xs,
-                                             OutputParams const &runparams,
-                                             ParagraphList::const_iterator par)
-{
-       switch (par->layout().latextype) {
-       case LATEX_COMMAND:
-               makeCommand(text, buf, xs, runparams, par);
-               break;
-       case LATEX_ENVIRONMENT:
-               makeEnvironment(text, buf, xs, runparams, par);
-               break;
-       case LATEX_LIST_ENVIRONMENT:
-       case LATEX_ITEM_ENVIRONMENT:
-               // Only case when makeAny() might consume more than one paragraph.
-               return makeListEnvironment(text, buf, xs, runparams, par);
-       case LATEX_PARAGRAPH:
-               makeParagraph(text, buf, xs, runparams, par);
-               break;
-       case LATEX_BIB_ENVIRONMENT:
-               makeBibliography(text, buf, xs, runparams, par);
-               break;
-       }
-       ++par;
-       return par;
-}
-
-
 bool isLayoutSectioning(Layout const & lay)
 {
        return lay.category() == from_utf8("Sectioning");
@@ -821,7 +764,7 @@ bool hasOnlyNotes(Paragraph const & par)
        for (int i = 0; i < par.size(); ++i)
                // If you find something that is not an inset (like actual text) or an inset that is not a note,
                // return false.
-               if (!par.isInset(i) || !dynamic_cast<InsetNote *>(par.insetList().get(i)))
+               if (!par.isInset(i) || par.getInset(i)->lyxCode() != NOTE_CODE)
                        return false;
        return true;
 }
@@ -851,7 +794,7 @@ DocBookInfoTag getParagraphsWithInfo(ParagraphList const &paragraphs,
        for (; cpit < epit; ++cpit) {
                // Skip paragraphs that don't generate anything in DocBook.
                Paragraph const & par = paragraphs[cpit];
-               if (par.empty() || par.emptyTag() || hasOnlyNotes(par))
+               if (hasOnlyNotes(par))
                        continue;
 
                // There should never be any section here. (Just a sanity check: if this fails, this function could end up
@@ -869,14 +812,14 @@ DocBookInfoTag getParagraphsWithInfo(ParagraphList const &paragraphs,
                }
 
                // Based on layout information, store this paragraph in one set: should be in <info>, must be,
-               // or abstract ().
+               // or abstract (either because of layout or of position).
                Layout const &style = par.layout();
 
                if (style.docbookininfo() == "always")
                        mustBeInInfo.emplace(cpit);
                else if (style.docbookininfo() == "maybe")
                        shouldBeInInfo.emplace(cpit);
-               else if (!hasAbstractLayout)
+               else if (documentHasSections && !hasAbstractLayout)
                        abstractNoLayout.emplace(cpit);
                else // This should definitely not be in <info>.
                        break;
@@ -892,6 +835,35 @@ DocBookInfoTag getParagraphsWithInfo(ParagraphList const &paragraphs,
 } // end anonymous namespace
 
 
+ParagraphList::const_iterator makeAny(Text const &text,
+                                      Buffer const &buf,
+                                      XMLStream &xs,
+                                      OutputParams const &runparams,
+                                      ParagraphList::const_iterator par)
+{
+       switch (par->layout().latextype) {
+       case LATEX_COMMAND:
+               makeCommand(text, buf, xs, runparams, par);
+               break;
+       case LATEX_ENVIRONMENT:
+               makeEnvironment(text, buf, xs, runparams, par);
+               break;
+       case LATEX_LIST_ENVIRONMENT:
+       case LATEX_ITEM_ENVIRONMENT:
+               // Only case when makeAny() might consume more than one paragraph.
+               return makeListEnvironment(text, buf, xs, runparams, par);
+       case LATEX_PARAGRAPH:
+               makeParagraph(text, buf, xs, runparams, par);
+               break;
+       case LATEX_BIB_ENVIRONMENT:
+               makeBibliography(text, buf, xs, runparams, par);
+               break;
+       }
+       ++par;
+       return par;
+}
+
+
 xml::FontTag docbookStartFontTag(xml::FontTypes type)
 {
        return xml::FontTag(from_utf8(fontToDocBookTag(type)), from_utf8(fontToAttribute(type)), type);
@@ -929,7 +901,7 @@ void outputDocBookInfo(
                // even though they must be properly output if there is some abstract.
                abstract = os2.str();
                docstring cleaned = abstract;
-               cleaned.erase(std::remove_if(cleaned.begin(), cleaned.end(), ::isspace), cleaned.end());
+               cleaned.erase(std::remove_if(cleaned.begin(), cleaned.end(), lyx::isSpace), cleaned.end());
 
                // Nothing? Then there is no abstract!
                if (cleaned.empty())
@@ -951,8 +923,16 @@ void outputDocBookInfo(
                // that mandating a wrapper like <info> would repel users. Thus, generate them first.
                makeAny(text, buf, xs, runparams, paragraphs.iterator_at(pit));
        for (auto pit : info.mustBeInInfo)
-               if (info.abstract.find(pit) == info.abstract.end()) // The abstract must be in info, but is dealt with after.
-                       makeAny(text, buf, xs, runparams, paragraphs.iterator_at(pit));
+               makeAny(text, buf, xs, runparams, paragraphs.iterator_at(pit));
+
+       // If there is no title, generate one (required for the document to be valid).
+       // This code is called for the main document, for table cells, etc., so be precise in this condition.
+       if (text.isMainText() && info.shouldBeInInfo.empty() && !runparams.inInclude) {
+               xs << xml::StartTag("title");
+               xs << "Untitled Document";
+               xs << xml::EndTag("title");
+               xs << xml::CR();
+       }
 
        // Always output the abstract as the last item of the <info>, as it requires special treatment (especially if
        // it contains several paragraphs that are empty).
@@ -978,6 +958,9 @@ void outputDocBookInfo(
 
        // End the <info> tag if it was started.
        if (needInfo) {
+               if (!xs.isLastTagCR())
+                       xs << xml::CR();
+
                xs << xml::EndTag("info");
                xs << xml::CR();
                xs.endDivision();
@@ -993,17 +976,17 @@ void docbookSimpleAllParagraphs(
 {
        // Handle the given text, supposing it has no sections (i.e. a "simple" text). The input may vary in length
        // between a single paragraph to a whole document.
+       pit_type const bpit = runparams.par_begin;
+       pit_type const epit = runparams.par_end;
+       ParagraphList const &paragraphs = text.paragraphs();
 
        // First, the <info> tag.
-       ParagraphList const &paragraphs = text.paragraphs();
-       pit_type bpit = runparams.par_begin;
-       pit_type const epit = runparams.par_end;
        DocBookInfoTag info = getParagraphsWithInfo(paragraphs, bpit, epit, false);
        outputDocBookInfo(text, buf, xs, runparams, paragraphs, info);
 
        // Then, the content. It starts where the <info> ends.
-       auto par = text.paragraphs().iterator_at(info.epit);
-       auto end = text.paragraphs().iterator_at(epit);
+       auto par = paragraphs.iterator_at(info.epit);
+       auto end = paragraphs.iterator_at(epit);
        while (par != end) {
                if (!hasOnlyNotes(*par))
                        par = makeAny(text, buf, xs, runparams, par);
@@ -1040,7 +1023,7 @@ void docbookParagraphs(Text const &text,
        tie(documentHasSections, eppit) = hasDocumentSectioning(paragraphs, bpit, epit);
 
        // Deal with "simple" documents, i.e. those without sections.
-       if (!documentHasSections){
+       if (!documentHasSections) {
                docbookSimpleAllParagraphs(text, buf, xs, runparams);
                return;
        }
@@ -1048,7 +1031,7 @@ void docbookParagraphs(Text const &text,
        // Output the first <info> tag (or just the title).
        DocBookInfoTag info = getParagraphsWithInfo(paragraphs, bpit, eppit, true);
        outputDocBookInfo(text, buf, xs, runparams, paragraphs, info);
-       bpit = eppit;
+       bpit = info.epit;
 
        // Then, iterate through the paragraphs of this document.
        bool currentlyInAppendix = false;
@@ -1121,10 +1104,9 @@ void docbookParagraphs(Text const &text,
 
                // Close all sections before the bibliography.
                // TODO: Only close all when the bibliography is at the end of the document? Or force to output the bibliography at the end of the document? Or don't care (as allowed by DocBook)?
-               auto insetsLength = distance(par->insetList().begin(), par->insetList().end());
-               if (insetsLength > 0) {
+               if (!par->insetList().empty()) {
                        Inset const *firstInset = par->getInset(0);
-                       if (firstInset && dynamic_cast<InsetBibtex const *>(firstInset)) {
+                       if (firstInset && (firstInset->lyxCode() == BIBITEM_CODE || firstInset->lyxCode() == BIBTEX_CODE)) {
                                while (!headerLevels.empty()) {
                                        int level = headerLevels.top().first;
                                        docstring tag = from_utf8("</" + headerLevels.top().second + ">");
@@ -1141,6 +1123,29 @@ void docbookParagraphs(Text const &text,
 
                // Generate this paragraph.
                par = makeAny(text, buf, xs, ourparams, par);
+
+               // Some special sections may require abstracts (mostly parts, in books).
+               // TODO: docbookforceabstracttag is a bit contrived here, but it does the job. Having another field just for this would be cleaner, but that's just for <part> and <partintro>, so it's probably not worth the effort.
+               if (isLayoutSectioning(style) && style.docbookforceabstracttag() != "NONE") {
+                       // This abstract may be found between the next paragraph and the next title.
+                       pit_type cpit = std::distance(text.paragraphs().begin(), par);
+                       pit_type ppit = std::get<1>(hasDocumentSectioning(paragraphs, cpit, epit));
+
+                       // Generate this abstract (this code corresponds to parts of outputDocBookInfo).
+                       DocBookInfoTag secInfo = getParagraphsWithInfo(paragraphs, cpit, ppit, true);
+
+                       if (!secInfo.abstract.empty()) {
+                               xs << xml::StartTag(style.docbookforceabstracttag());
+                               xs << xml::CR();
+                               for (auto const &p : secInfo.abstract)
+                                       makeAny(text, buf, xs, runparams, paragraphs.iterator_at(p));
+                               xs << xml::EndTag(style.docbookforceabstracttag());
+                               xs << xml::CR();
+                       }
+
+                       // Skip all the text that just has been generated.
+                       par = paragraphs.iterator_at(ppit);
+               }
        }
 
        // If need be, close <section>s, but only at the end of the document (otherwise, dealt with at the beginning