• Bug report: major / minor options are swapped

  • We tried to make OpenKM as intuitive as possible, but an advice is always welcome.
We tried to make OpenKM as intuitive as possible, but an advice is always welcome.
Forum rules: Please, before asking something see the documentation wiki or use the search feature of the forum. And remember we don't have a crystal ball or mental readers, so if you post about an issue tell us which OpenKM are you using and also the browser and operating system version. For more info read How to Report Bugs Effectively.
 #52613  by ryanm
 
Hi,

We're running OpenKM CE 6.3.9 in Docker (openkm/openkm-ce:6.3.9) with version.numeration.adapter= com.openkm.vernum.MajorMinorReleaseVersionNumerationAdapter

When we update documents and check the "Increment minor version" box, the major version is incremented. And vice versa.

I think I've found the issue in the source:

In src/main/java/com/openkm/vernum/MajorMinorReleaseVersionNumerationAdapter.java
Code: Select all
public static final int MINOR = 1;
public static final int MAJOR = 2;
This is reversed compared with src/main/java/com/openkm/frontend/client/widget/upload/FancyFileUpload.java ...
Code: Select all
if (increaseMajorVersion.getValue()) {
	uploadForm.setIncreaseVersion(1);
} else if (increaseMinorVersion.getValue()) {
	uploadForm.setIncreaseVersion(2);
} else {
	uploadForm.setIncreaseVersion(0);
}
... and src/main/java/com/openkm/extension/frontend/client/widget/htmleditor/CheckinPopup.java
Code: Select all
int increaseVersion = 0;
if (increaseMajorVersion.getValue()) {
	increaseVersion = 1;
} else if (increaseMinorVersion.getValue()) {
	increaseVersion = 2;
}
Thanks,
Ryan

About Us

OpenKM is part of the management software. A management software is a program that facilitates the accomplishment of administrative tasks. OpenKM is a document management system that allows you to manage business content and workflow in a more efficient way. Document managers guarantee data protection by establishing information security for business content.