Limit client sku association to active clients

jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/ClientMapper.java

@@ -20,6 +20,7 @@ public interface ClientMapper extends BaseMapper<Client> {
     String getClientEntity(@Param("id") String id);
     Map<String, String> getClientsEntity(@Param("ids") List<String> ids);
     String getClientIdByCode(@Param("code") String code);
+    String getActiveClientIdByCode(@Param("code") String code);
     List<Client> getClientByType(@Param("type") String type);
     Client getClientByCode(@Param("code") String internalCode);
     Client getClientFromOrder(@Param("orderId")String orderId);

jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/xml/ClientMapper.xml

@@ -23,6 +23,11 @@
     SELECT *
     FROM client
         WHERE internal_code = #{code}
+</select>
+    <select id="getActiveClientByCode" parameterType="java.lang.String" resultType="org.jeecg.modules.business.entity.Client">
+        SELECT *
+        FROM client
+        WHERE internal_code = #{code} AND active = 1
     </select>
     <select id="getClientByType" resultType="org.jeecg.modules.business.entity.Client">
         SELECT c.*

jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/IClientService.java

@@ -41,6 +41,7 @@ public interface IClientService extends IService<Client> {
 	public String getClientEntity(String id);
 	public Map<String, String> getClientsEntity(List<String> ids);
 	public String getClientIdByCode(String code);
+	public String getActiveClientIdByCode(String code);
 	/**
 	 * Get current user's client information
 	 * @return client or null if current user's role is not client

jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/ClientServiceImpl.java

@@ -161,6 +161,10 @@ public class ClientServiceImpl extends ServiceImpl<ClientMapper, Client> impleme
     public String getClientIdByCode(String code) {
         return clientMapper.getClientIdByCode(code);
     }
+    @Override
+    public String getActiveClientIdByCode(String code) {
+        return clientMapper.getActiveClientIdByCode(code);
+    }
 
     @Override
     public void anonymizePersonalData(int directClientAnonymizationPeriod) {

jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/ClientSkuServiceImpl.java

@@ -49,7 +49,7 @@ public class ClientSkuServiceImpl extends ServiceImpl<ClientSkuMapper, ClientSku
 				continue;
 			}
 			String internalCode = erpCode.substring(index+1);
-			String clientId = clientService.getClientIdByCode(internalCode);
+			String clientId = clientService.getActiveClientIdByCode(internalCode);
             if (clientId != null) {
 				log.info("Associating sku \"{}\" with client \"{}\" : \"{}\". ", sku.getErpCode(), internalCode, clientId);
 				addClientSku(clientId, sku.getId());

jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/SkuListMabangServiceImpl.java

@@ -410,7 +410,7 @@ public class SkuListMabangServiceImpl extends ServiceImpl<SkuListMabangMapper, S
                     s.setZhName(skuData.getNameCN());
                 }
             }
-            if(skuData.getStockPicture().equals("") || skuData.getStockPicture() == null) {
+            if(skuData.getStockPicture() == null || skuData.getStockPicture().isEmpty()) {
                 s.setImageSource(skuData.getSalePicture());
             }
             else {