From ae68147d0b6781a92ef423650cb6b44ed0149f72 Mon Sep 17 00:00:00 2001
From: Yann Diorcet <diorcet.yann@gmail.com>
Date: Mon, 28 Dec 2020 18:24:50 +0100
Subject: [PATCH] LinearPattern: Support 1 occurrence

When using linear pattern with expression, this can be usefull to have only one
occurence (the original one), which can't be done with the following
implemenation. This commit change this behaviour by allowing this value.
---
 src/Mod/PartDesign/App/FeatureLinearPattern.cpp | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/Mod/PartDesign/App/FeatureLinearPattern.cpp b/src/Mod/PartDesign/App/FeatureLinearPattern.cpp
index adc2918ab3..463339fd2b 100644
--- a/src/Mod/PartDesign/App/FeatureLinearPattern.cpp
+++ b/src/Mod/PartDesign/App/FeatureLinearPattern.cpp
@@ -73,12 +73,10 @@ const std::list<gp_Trsf> LinearPattern::getTransformations(const std::vector<App
     if (distance < Precision::Confusion())
         throw Base::ValueError("Pattern length too small");
     int occurrences = Occurrences.getValue();
-    if (occurrences < 2)
-        throw Base::ValueError("At least two occurrences required");
+    if (occurrences < 1)
+        throw Base::ValueError("At least one occurrence required");
     bool reversed = Reversed.getValue();
 
-    double offset = distance / (occurrences - 1);
-
     App::DocumentObject* refObject = Direction.getValue();
     if (refObject == NULL)
         throw Base::ValueError("No direction reference specified");
@@ -178,9 +176,12 @@ const std::list<gp_Trsf> LinearPattern::getTransformations(const std::vector<App
     gp_Trsf trans;
     transformations.push_back(trans); // identity transformation
 
-    for (int i = 1; i < occurrences; i++) {
-        trans.SetTranslation(direction * i * offset);
-        transformations.push_back(trans);
+    if (occurrences > 1) {
+        double offset = distance / (occurrences - 1);
+        for (int i = 1; i < occurrences; i++) {
+            trans.SetTranslation(direction * i * offset);
+            transformations.push_back(trans);
+        }
     }
 
     return transformations;