}
+bool BufferParams::removeBadModules()
+{
+ // we'll write a new list of modules, since we can't just remove them,
+ // as that would invalidate our iterators
+ list<string> oldModules = getModules();
+ clearLayoutModules();
+
+ list<string> const & provmods = baseClass()->providedModules();
+ list<string> const & exclmods = baseClass()->excludedModules();
+ bool consistent = true; // set to false if we have to do anything
+
+ list<string>::const_iterator oit = oldModules.begin();
+ list<string>::const_iterator const oen = oldModules.end();
+ for (; oit != oen; ++oit) {
+ string const & modname = *oit;
+ // skip modules that the class provides
+ if (find(provmods.begin(), provmods.end(), modname) != provmods.end()) {
+ LYXERR0("Module `" << modname << "' dropped because provided by document class.");
+ consistent = false;
+ continue;
+ }
+ // are we excluded by the document class?
+ if (find(exclmods.begin(), exclmods.end(), modname) != exclmods.end()) {
+ LYXERR0("Module `" << modname << "' dropped because excluded by document class.");
+ consistent = false;
+ continue;
+ }
+ // determine whether some provided module excludes us or we exclude it
+ list<string>::const_iterator pit = provmods.begin();
+ list<string>::const_iterator const pen = provmods.end();
+ bool excluded = false;
+ for (; !excluded && pit != pen; ++pit) {
+ if (!LyXModule::areCompatible(modname, *pit)) {
+ LYXERR0("Module " << modname <<
+ " dropped becuase it conflicts with provided module `" << *pit << "'.");
+ consistent = false;
+ excluded = true;
+ }
+ }
+ if (excluded)
+ continue;
+ layoutModules_.push_back(modname);
+ }
+ return consistent;
+}
+
+
void BufferParams::addDefaultModules()
{
// add any default modules not already in use
list<string>::const_iterator mit = mods.begin();
list<string>::const_iterator men = mods.end();
- // we want to add these to the front, but in the right order,
- // so we collect them here first.
- list<string> modulesToAdd;
+ // We want to insert the default modules at the beginning of
+ // the list, but also to insert them in the correct order.
+ // The obvious thing to do would be to collect them and then
+ // insert them, but that doesn't work because a later default
+ // module may require an earlier one, and then the test below
+ // moduleCanBeAdded(modname)
+ // will fail. So we have to do it a more complicated way.
+ list<string>::iterator insertpos = layoutModules_.begin();
+ int numinserts = 0;
for (; mit != men; mit++) {
string const & modName = *mit;
continue;
}
- if (moduleCanBeAdded(modName)) {
- LYXERR(Debug::TCLASS, "Default module `" << modName << "' added.");
- modulesToAdd.push_back(modName);
- } else
- LYXERR(Debug::TCLASS,
- "Default module `" << modName << "' could not be added.");
+ if (!moduleCanBeAdded(modName)) {
+ // FIXME This could be because it's already present, so we should
+ // probably return something indicating that.
+ LYXERR(Debug::TCLASS, "Default module `" << modName <<
+ "' could not be added.");
+ continue;
+ }
+ LYXERR(Debug::TCLASS, "Default module `" << modName << "' added.");
+ layoutModules_.insert(insertpos, modName);
+ // now we reset insertpos
+ ++numinserts;
+ insertpos = layoutModules_.begin();
+ advance(insertpos, numinserts);
}
-
- // OK, now we can add the default modules.
- layoutModules_.insert(
- layoutModules_.begin(), modulesToAdd.begin(), modulesToAdd.end());
}
bool BufferParams::checkModuleConsistency() {
bool consistent = true;
- // Perform a consistency check on the set of modules.
- // In particular, we need to check that modules provided by this class
- // do not conflict with modules chosen by the user.
+ // Perform a consistency check on the set of modules. We need to make
+ // sure that none of the modules exclude each other and that requires
+ // are satisfied.
list<string> oldModules = getModules();
clearLayoutModules();
list<string>::const_iterator oit = oldModules.begin();
list<string> const & exclmods = baseClass()->excludedModules();
for (; oit != oen; ++oit) {
string const & modname = *oit;
- // skip modules that the class provides
- if (find(provmods.begin(), provmods.end(), modname) != provmods.end()) {
- consistent = false;
- LYXERR0("Module " << modname << " dropped because provided by document class.");
- continue;
- }
- // are we excluded by the document class?
- if (find(exclmods.begin(), exclmods.end(), modname) != exclmods.end()) {
- consistent = false;
- LYXERR0("Module " << modname << " dropped because excluded by document class.");
- continue;
- }
-
- // determine whether some provided module excludes us or we exclude it
- list<string>::const_iterator pit = provmods.begin();
- list<string>::const_iterator pen = provmods.end();
bool excluded = false;
- for (; !excluded && pit != pen; ++pit) {
- if (!LyXModule::areCompatible(modname, *pit)) {
- consistent = false;
- LYXERR0("Module " << modname <<
- " dropped becuase it conflicts with provided module " << *pit);
- excluded = true;
- }
- }
-
- if (excluded)
- continue;
-
// Determine whether some prior module excludes us, or we exclude it
list<string>::const_iterator lit = layoutModules_.begin();
list<string>::const_iterator len = layoutModules_.end();
}
pimpl_->baseClass_ = classname;
+ // the previous document class may have loaded some modules that the
+ // new one excludes, and the new class may provide, etc, some that
+ // conflict with ones that were already loaded. So we need to go
+ // through the list and fix everything. I suppose there are various
+ // ways this could be done, but the following seems to work at the
+ // moment. (Thanks to Philippe Charpentier for helping work out all
+ // the bugs---rgh.)
+ //
+ // first, we remove any modules the new document class itself provides,
+ // those it excludes, and those that conflict with ones it excludes.
+ // this has to be done first because, otherwise, a module we're about
+ // to remove could prevent a default module from being added.
+ removeBadModules();
+ // next, we add any default modules the new class provides.
addDefaultModules();
+ // finally, we perform a general consistency check on the set of
+ // loaded modules.
checkModuleConsistency();
+ // FIXME removeBadModules() and checkModuleConsistency() both return
+ // a boolean indicating whether something had to be changed. It might
+ // be worth popping a message to the user if so.
return true;
}
projects / lyx.git / blobdiff