@univerjs/sheets-data-validation 1.0.0-rc.0 → 1.0.1

package/lib/cjs/facade.js

@@ -7,7 +7,7 @@ let _univerjs_engine_formula = require("@univerjs/engine-formula");
 let _univerjs_sheets = require("@univerjs/sheets");
 let _univerjs_core_facade = require("@univerjs/core/facade");
 
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/typeof.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/typeof.js
 function _typeof(o) {
 	"@babel/helpers - typeof";
 	return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function(o) {
@@ -18,7 +18,7 @@ function _typeof(o) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPrimitive.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPrimitive.js
 function toPrimitive(t, r) {
 	if ("object" != _typeof(t) || !t) return t;
 	var e = t[Symbol.toPrimitive];
@@ -31,14 +31,14 @@ function toPrimitive(t, r) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPropertyKey.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPropertyKey.js
 function toPropertyKey(t) {
 	var i = toPrimitive(t, "string");
 	return "symbol" == _typeof(i) ? i : i + "";
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/defineProperty.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/defineProperty.js
 function _defineProperty(e, r, t) {
 	return (r = toPropertyKey(r)) in e ? Object.defineProperty(e, r, {
 		value: t,
@@ -153,7 +153,7 @@ var FDataValidationBuilder = class FDataValidationBuilder {
 	* ```
 	*/
 	getAllowInvalid() {
-		return this._rule.errorStyle !== _univerjs_core.DataValidationErrorStyle.STOP;
+		return this._rule.showErrorMessage === false || this._rule.errorStyle !== _univerjs_core.DataValidationErrorStyle.STOP;
 	}
 	/**
 	* Gets the data validation type of the rule
@@ -961,7 +961,14 @@ var FDataValidationBuilder = class FDataValidationBuilder {
 */
 var FDataValidation = class {
 	constructor(rule, worksheet, _injector) {
-		_defineProperty(this, "rule", void 0);
+		_defineProperty(
+			this,
+			/**
+			* The underlying validation rule data. Use the Facade setters to update a rule attached to a worksheet.
+			*/
+			"rule",
+			void 0
+		);
 		_defineProperty(this, "_worksheet", void 0);
 		_defineProperty(this, "_injector", void 0);
 		this._injector = _injector;
@@ -983,7 +990,7 @@ var FDataValidation = class {
 	* ```
 	*/
 	getAllowInvalid() {
-		return this.rule.errorStyle !== _univerjs_core.DataValidationErrorStyle.STOP;
+		return this.rule.showErrorMessage === false || this.rule.errorStyle !== _univerjs_core.DataValidationErrorStyle.STOP;
 	}
 	/**
 	* Gets the data validation type of the rule
@@ -1129,8 +1136,8 @@ var FDataValidation = class {
 		return this.rule.ranges.map((range) => this._injector.createInstance(_univerjs_sheets_facade.FRange, workbook, this._worksheet, range));
 	}
 	/**
-	* Gets the unit ID of the worksheet
-	* @returns {string | undefined} The unit ID of the worksheet
+	* Gets the unit ID of the workbook
+	* @returns {string | undefined} The unit ID of the workbook
 	* @example
 	* ```typescript
 	* const fWorkbook = univerAPI.getActiveWorkbook();
@@ -1360,7 +1367,7 @@ var FRangeSheetsDataValidationMixin = class extends _univerjs_sheets_facade.FRan
 		const worksheet = this._worksheet;
 		const sheetName = worksheet.getName();
 		const errors = [];
-		for (const range of ranges) {
+		for (const range of (0, _univerjs_sheets_data_validation.getRangesInWorksheet)(ranges, worksheet)) {
 			const promises = [];
 			for (let row = range.startRow; row <= range.endRow; row++) for (let col = range.startColumn; col <= range.endColumn; col++) promises.push((async () => {
 				try {
@@ -1600,7 +1607,7 @@ var FWorkbookSheetsDataValidationMixin = class extends _univerjs_sheets_facade.F
 		if (!worksheet) throw new Error(`Cannot find worksheet with sheetId: ${sheetId}`);
 		const sheetName = worksheet.getName();
 		const errors = [];
-		for (const range of ranges) {
+		for (const range of (0, _univerjs_sheets_data_validation.getRangesInWorksheet)(ranges, worksheet)) {
 			const promises = [];
 			for (let row = range.startRow; row <= range.endRow; row++) for (let col = range.startColumn; col <= range.endColumn; col++) promises.push((async () => {
 				try {
@@ -1668,7 +1675,7 @@ var FWorksheetDataValidationMixin = class extends _univerjs_sheets_facade.FWorks
 		const worksheet = this._worksheet;
 		const sheetName = worksheet.getName();
 		const errors = [];
-		for (const range of ranges) {
+		for (const range of (0, _univerjs_sheets_data_validation.getRangesInWorksheet)(ranges, worksheet)) {
 			const promises = [];
 			for (let row = range.startRow; row <= range.endRow; row++) for (let col = range.startColumn; col <= range.endColumn; col++) promises.push((async () => {
 				try {

package/lib/cjs/index.js

@@ -7,7 +7,36 @@ let rxjs = require("rxjs");
 let _univerjs_protocol = require("@univerjs/protocol");
 let _univerjs_sheets_formula = require("@univerjs/sheets-formula");
 
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/typeof.js
+//#region src/utils/range.ts
+function getRangeInWorksheet(range, worksheet) {
+	const maxRows = worksheet.getMaxRows();
+	const maxColumns = worksheet.getMaxColumns();
+	if (maxRows <= 0 || maxColumns <= 0) return null;
+	const effectiveRange = (0, _univerjs_core.getIntersectRange)(_univerjs_core.Range.transformRange(range, worksheet), {
+		startRow: 0,
+		endRow: maxRows - 1,
+		startColumn: 0,
+		endColumn: maxColumns - 1
+	});
+	if (!effectiveRange) return null;
+	return {
+		startRow: effectiveRange.startRow,
+		endRow: effectiveRange.endRow,
+		startColumn: effectiveRange.startColumn,
+		endColumn: effectiveRange.endColumn
+	};
+}
+function getRangesInWorksheet(ranges, worksheet) {
+	const effectiveRanges = [];
+	ranges.forEach((range) => {
+		const effectiveRange = getRangeInWorksheet(range, worksheet);
+		if (effectiveRange) effectiveRanges.push(effectiveRange);
+	});
+	return effectiveRanges;
+}
+
+//#endregion
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/typeof.js
 function _typeof(o) {
 	"@babel/helpers - typeof";
 	return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function(o) {
@@ -18,7 +47,7 @@ function _typeof(o) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPrimitive.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPrimitive.js
 function toPrimitive(t, r) {
 	if ("object" != _typeof(t) || !t) return t;
 	var e = t[Symbol.toPrimitive];
@@ -31,14 +60,14 @@ function toPrimitive(t, r) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPropertyKey.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPropertyKey.js
 function toPropertyKey(t) {
 	var i = toPrimitive(t, "string");
 	return "symbol" == _typeof(i) ? i : i + "";
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/defineProperty.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/defineProperty.js
 function _defineProperty(e, r, t) {
 	return (r = toPropertyKey(r)) in e ? Object.defineProperty(e, r, {
 		value: t,
@@ -49,7 +78,7 @@ function _defineProperty(e, r, t) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/decorateParam.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/decorateParam.js
 function __decorateParam(paramIndex, decorator) {
 	return function(target, key) {
 		decorator(target, key, paramIndex);
@@ -57,7 +86,7 @@ function __decorateParam(paramIndex, decorator) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/decorate.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/decorate.js
 function __decorate(decorators, target, key, desc) {
 	var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
 	if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
@@ -133,9 +162,15 @@ let DataValidationCacheService = class DataValidationCacheService extends _unive
 	removeRule(unitId, subUnitId, rule) {
 		this._deleteRange(unitId, subUnitId, rule.ranges);
 	}
+	_getEffectiveRanges(unitId, subUnitId, ranges) {
+		var _this$_univerInstance;
+		const worksheet = (_this$_univerInstance = this._univerInstanceService.getUnit(unitId, _univerjs_core.UniverInstanceType.UNIVER_SHEET)) === null || _this$_univerInstance === void 0 ? void 0 : _this$_univerInstance.getSheetBySheetId(subUnitId);
+		if (!worksheet) return [];
+		return getRangesInWorksheet(ranges, worksheet);
+	}
 	markRangeDirty(unitId, subUnitId, ranges, isSetRange) {
 		const cache = this._ensureCache(unitId, subUnitId);
-		ranges.forEach((range) => {
+		this._getEffectiveRanges(unitId, subUnitId, ranges).forEach((range) => {
 			_univerjs_core.Range.foreach(range, (row, col) => {
 				if (cache.getValue(row, col) !== void 0) cache.setValue(row, col, void 0);
 			});
@@ -149,7 +184,7 @@ let DataValidationCacheService = class DataValidationCacheService extends _unive
 	}
 	_deleteRange(unitId, subUnitId, ranges) {
 		const cache = this._ensureCache(unitId, subUnitId);
-		ranges.forEach((range) => {
+		this._getEffectiveRanges(unitId, subUnitId, ranges).forEach((range) => {
 			_univerjs_core.Range.foreach(range, (row, col) => {
 				cache.realDeleteValue(row, col);
 			});
@@ -286,11 +321,16 @@ let DataValidationCustomFormulaService = class DataValidationCustomFormulaServic
 		}
 	}
 	_addFormulaByRange(unitId, subUnitId, ruleId, formula, formula2, ranges) {
+		var _this$_instanceSrv$ge;
 		const { ruleFormulaMap, ruleFormulaMap2 } = this._ensureMaps(unitId, subUnitId);
-		const originRow = ranges[0].startRow;
-		const originCol = ranges[0].startColumn;
+		const worksheet = (_this$_instanceSrv$ge = this._instanceSrv.getUnit(unitId, _univerjs_core.UniverInstanceType.UNIVER_SHEET)) === null || _this$_instanceSrv$ge === void 0 ? void 0 : _this$_instanceSrv$ge.getSheetBySheetId(subUnitId);
+		if (!worksheet) return;
+		const effectiveRanges = getRangesInWorksheet(ranges, worksheet);
+		if (!effectiveRanges.length) return;
+		const originRow = effectiveRanges[0].startRow;
+		const originCol = effectiveRanges[0].startColumn;
 		if (formula && (0, _univerjs_core.isFormulaString)(formula)) {
-			const formulaId = this._registerFormula(unitId, subUnitId, ruleId, formula, ranges);
+			const formulaId = this._registerFormula(unitId, subUnitId, ruleId, formula, effectiveRanges);
 			ruleFormulaMap.set(ruleId, {
 				formula,
 				originCol,
@@ -299,7 +339,7 @@ let DataValidationCustomFormulaService = class DataValidationCustomFormulaServic
 			});
 		}
 		if (formula2 && (0, _univerjs_core.isFormulaString)(formula2)) {
-			const formulaId2 = this._registerFormula(unitId, subUnitId, ruleId, formula2, ranges);
+			const formulaId2 = this._registerFormula(unitId, subUnitId, ruleId, formula2, effectiveRanges);
 			ruleFormulaMap2.set(ruleId, {
 				formula: formula2,
 				originCol,
@@ -838,9 +878,7 @@ let SheetDataValidationModel = class SheetDataValidationModel extends _univerjs_
 				case "update":
 					this._updateRule(ruleChange.unitId, ruleChange.subUnitId, ruleChange.rule.uid, ruleChange.oldRule, ruleChange.updatePayload);
 					break;
-				case "remove":
-					this._removeRule(ruleChange.unitId, ruleChange.subUnitId, ruleChange.rule);
-					break;
+				case "remove": this._removeRule(ruleChange.unitId, ruleChange.subUnitId, ruleChange.rule);
 			}
 			this._ruleChange$.next(ruleChange);
 		}));
@@ -1552,7 +1590,8 @@ var ListValidator = class extends BaseSheetValidator {
 		return { tb };
 	}
 	parseCellValue(cellValue) {
-		return (0, _univerjs_sheets.deserializeListOptions)(cellValue.toString());
+		const cellString = cellValue.toString();
+		return (0, _univerjs_sheets.deserializeListOptions)(cellString);
 	}
 	async parseFormula(rule, unitId, subUnitId) {
 		var _results$;
@@ -1703,7 +1742,8 @@ function getDataValidationDiffMutations(unitId, subUnitId, diffs, accessor, sour
 	const redoMutations = [];
 	const undoMutations = [];
 	const sheetDataValidationModel = accessor.get(SheetDataValidationModel);
-	const target = (0, _univerjs_sheets.getSheetCommandTarget)(accessor.get(_univerjs_core.IUniverInstanceService), {
+	const univerInstanceService = accessor.get(_univerjs_core.IUniverInstanceService);
+	const target = (0, _univerjs_sheets.getSheetCommandTarget)(univerInstanceService, {
 		unitId,
 		subUnitId
 	});
@@ -1716,7 +1756,7 @@ function getDataValidationDiffMutations(unitId, subUnitId, diffs, accessor, sour
 	let setRangeValue = false;
 	function setRangesDefaultValue(ranges, defaultValue) {
 		if (!fillDefaultValue) return;
-		ranges.forEach((range) => {
+		getRangesInWorksheet(ranges, worksheet).forEach((range) => {
 			_univerjs_core.Range.foreach(range, (row, column) => {
 				const cellData = worksheet.getCellRaw(row, column);
 				const value = getStringCellValue(cellData);
@@ -1882,8 +1922,6 @@ function getDataValidationDiffMutations(unitId, subUnitId, diffs, accessor, sour
 					const formula = sheetDataValidationModel.getValidator(_univerjs_core.DataValidationType.CHECKBOX).parseFormulaSync(diff.rule, unitId, subUnitId);
 					setRangesDefaultValue(diff.rule.ranges, formula.originFormula2);
 				}
-				break;
-			default: break;
 		}
 	});
 	if (setRangeValue) {
@@ -2022,7 +2060,8 @@ const UpdateSheetDataValidationSettingCommand = {
 		}];
 		if (setting.type === _univerjs_core.DataValidationType.CHECKBOX) {
 			const ranges = rule.ranges;
-			const target = (0, _univerjs_sheets.getSheetCommandTarget)(accessor.get(_univerjs_core.IUniverInstanceService), {
+			const univerInstanceService = accessor.get(_univerjs_core.IUniverInstanceService);
+			const target = (0, _univerjs_sheets.getSheetCommandTarget)(univerInstanceService, {
 				unitId,
 				subUnitId
 			});
@@ -2032,7 +2071,7 @@ const UpdateSheetDataValidationSettingCommand = {
 				const { formula2: oldFormula2 = 0, formula1: oldFormula1 = 1 } = rule;
 				const { formula2 = 0, formula1 = 1 } = setting;
 				let setted = false;
-				ranges.forEach((range) => {
+				getRangesInWorksheet(ranges, worksheet).forEach((range) => {
 					_univerjs_core.Range.foreach(range, (row, column) => {
 						const cellData = worksheet.getCellRaw(row, column);
 						const value = getStringCellValue(cellData);
@@ -2133,7 +2172,8 @@ const ClearRangeDataValidationCommand = {
 		if (!params) return false;
 		const { unitId, subUnitId, ranges, types } = params;
 		const commandService = accessor.get(_univerjs_core.ICommandService);
-		const target = (0, _univerjs_sheets.getSheetCommandTarget)(accessor.get(_univerjs_core.IUniverInstanceService), {
+		const univerInstanceService = accessor.get(_univerjs_core.IUniverInstanceService);
+		const target = (0, _univerjs_sheets.getSheetCommandTarget)(univerInstanceService, {
 			unitId,
 			subUnitId
 		});
@@ -2291,7 +2331,9 @@ let DataValidationFormulaController = class DataValidationFormulaController exte
 				if (!this._permissionService.getPermissionPoint(new _univerjs_sheets.WorksheetViewPermission(unitId, subUnitId).id)) return false;
 			}
 			if (!targetSheet) return false;
-			const { startRow, endRow, startColumn, endColumn } = sequenceGrid.range;
+			const range = getRangeInWorksheet(sequenceGrid.range, targetSheet);
+			if (!range) continue;
+			const { startRow, endRow, startColumn, endColumn } = range;
 			for (let i = startRow; i <= endRow; i++) for (let j = startColumn; j <= endColumn; j++) {
 				var _targetSheet$getCell;
 				const permission = (_targetSheet$getCell = targetSheet.getCell(i, j)) === null || _targetSheet$getCell === void 0 || (_targetSheet$getCell = _targetSheet$getCell.selectionProtection) === null || _targetSheet$getCell === void 0 ? void 0 : _targetSheet$getCell[0];
@@ -2310,7 +2352,7 @@ DataValidationFormulaController = __decorate([
 //#endregion
 //#region package.json
 var name = "@univerjs/sheets-data-validation";
-var version = "1.0.0-rc.0";
+var version = "1.0.1";
 
 //#endregion
 //#region src/config/config.ts
@@ -3023,7 +3065,8 @@ let DataValidationController = class DataValidationController extends _univerjs_
 				const selections = (_this$_selectionManag = this._selectionManagerService.getCurrentSelections()) === null || _this$_selectionManag === void 0 ? void 0 : _this$_selectionManag.map((s) => s.range);
 				const ruleMatrix = this._sheetDataValidationModel.getRuleObjectMatrix(unitId, subUnitId).clone();
 				if (selections) ruleMatrix.removeRange(selections);
-				const { redoMutations, undoMutations } = getDataValidationDiffMutations(unitId, subUnitId, ruleMatrix.diff(this._sheetDataValidationModel.getRules(unitId, subUnitId)), this._injector, "patched");
+				const diffs = ruleMatrix.diff(this._sheetDataValidationModel.getRules(unitId, subUnitId));
+				const { redoMutations, undoMutations } = getDataValidationDiffMutations(unitId, subUnitId, diffs, this._injector, "patched");
 				return {
 					undos: undoMutations,
 					redos: redoMutations
@@ -3061,14 +3104,14 @@ let SheetsDataValidationValidatorService = class SheetsDataValidationValidatorSe
 			if (ranges.length === 0) return;
 			const workbook = this._univerInstanceService.getCurrentUnitOfType(_univerjs_core.UniverInstanceType.UNIVER_SHEET);
 			const worksheet = workbook === null || workbook === void 0 ? void 0 : workbook.getActiveSheet();
-			const map = {};
+			const map = Object.create(null);
 			ranges.flat().forEach((range) => {
-				if (!map[range.unitId]) map[range.unitId] = {};
+				if (!map[range.unitId]) map[range.unitId] = Object.create(null);
 				if (!map[range.unitId][range.subUnitId]) map[range.unitId][range.subUnitId] = [];
 				const workbook = this._univerInstanceService.getUnit(range.unitId, _univerjs_core.UniverInstanceType.UNIVER_SHEET);
 				const worksheet = workbook === null || workbook === void 0 ? void 0 : workbook.getSheetBySheetId(range.subUnitId);
 				if (!worksheet) return;
-				map[range.unitId][range.subUnitId].push(...range.ranges.map((range) => _univerjs_core.Range.transformRange(range, worksheet)));
+				map[range.unitId][range.subUnitId].push(...getRangesInWorksheet(range.ranges, worksheet));
 			});
 			Object.entries(map).forEach(([unitId, subUnitMap]) => {
 				Object.entries(subUnitMap).forEach(([subUnitId, ranges]) => {
@@ -3121,11 +3164,10 @@ let SheetsDataValidationValidatorService = class SheetsDataValidationValidatorSe
 		if (!workbook) throw new Error(`cannot find current workbook, unitId: ${unitId}`);
 		const worksheet = workbook.getSheetBySheetId(subUnitId);
 		if (!worksheet) throw new Error(`cannot find current worksheet, sheetId: ${subUnitId}`);
-		const allRules = this._sheetDataValidationModel.getRules(unitId, subUnitId);
-		const ruleRanges = [];
-		for (const rule of allRules) ruleRanges.push(...rule.ranges);
+		const effectiveRanges = getRangesInWorksheet(ranges, worksheet);
+		const ruleRanges = getRangesInWorksheet(this._sheetDataValidationModel.getRules(unitId, subUnitId).flatMap((rule) => rule.ranges), worksheet);
 		const intersectRanges = [];
-		for (const range of ranges) for (const ruleRange of ruleRanges) {
+		for (const range of effectiveRanges) for (const ruleRange of ruleRanges) {
 			const intersect = (0, _univerjs_core.getIntersectRange)(range, ruleRange);
 			if (intersect) intersectRanges.push(intersect);
 		}
@@ -3158,31 +3200,32 @@ let SheetsDataValidationValidatorService = class SheetsDataValidationValidatorSe
 		if (!workbook) throw new Error(`cannot find current workbook, unitId: ${unitId}`);
 		const worksheet = workbook.getSheetBySheetId(subUnitId);
 		if (!worksheet) throw new Error(`cannot find current worksheet, sheetId: ${subUnitId}`);
-		const rules = this._sheetDataValidationModel.getRules(unitId, subUnitId);
-		await Promise.all(rules.map((rule) => {
-			return Promise.all(rule.ranges.map((range) => {
-				const promises = [];
-				_univerjs_core.Range.foreach(range, (row, col) => {
-					promises.push(this._validatorByCell(workbook, worksheet, row, col));
-				});
-				return Promise.all(promises);
-			}));
+		const ranges = getRangesInWorksheet(this._sheetDataValidationModel.getRules(unitId, subUnitId).flatMap((rule) => rule.ranges), worksheet);
+		await Promise.all(ranges.map((range) => {
+			const promises = [];
+			_univerjs_core.Range.foreach(range, (row, col) => {
+				promises.push(this._validatorByCell(workbook, worksheet, row, col));
+			});
+			return Promise.all(promises);
 		}));
 		return this._dataValidationCacheService.ensureCache(unitId, subUnitId);
 	}
 	async validatorWorkbook(unitId) {
 		const sheetIds = this._sheetDataValidationModel.getSubUnitIds(unitId);
 		const results = await Promise.all(sheetIds.map((id) => this.validatorWorksheet(unitId, id)));
-		const map = {};
+		const map = Object.create(null);
 		results.forEach((result, i) => {
 			map[sheetIds[i]] = result;
 		});
 		return map;
 	}
 	getDataValidations(unitId, subUnitId, ranges) {
+		var _this$_univerInstance;
+		const worksheet = (_this$_univerInstance = this._univerInstanceService.getUnit(unitId, _univerjs_core.UniverInstanceType.UNIVER_SHEET)) === null || _this$_univerInstance === void 0 ? void 0 : _this$_univerInstance.getSheetBySheetId(subUnitId);
+		if (!worksheet) return [];
 		const ruleMatrix = this._sheetDataValidationModel.getRuleObjectMatrix(unitId, subUnitId);
 		const ruleIdSet = /* @__PURE__ */ new Set();
-		ranges.forEach((range) => {
+		getRangesInWorksheet(ranges, worksheet).forEach((range) => {
 			_univerjs_core.Range.foreach(range, (row, col) => {
 				const ruleId = ruleMatrix.getValue(row, col);
 				if (ruleId) ruleIdSet.add(ruleId);
@@ -3368,6 +3411,7 @@ exports.getDataValidationCellValue = getDataValidationCellValue;
 exports.getDataValidationDiffMutations = getDataValidationDiffMutations;
 exports.getFormulaCellData = getFormulaCellData;
 exports.getFormulaResult = getFormulaResult;
+exports.getRangesInWorksheet = getRangesInWorksheet;
 exports.getTransformedFormula = getTransformedFormula;
 exports.isLegalFormulaResult = isLegalFormulaResult;
 exports.transformCheckboxValue = transformCheckboxValue;

package/lib/cjs/locale/ar-SA.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/ar-SA.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/ca-ES.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/ca-ES.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/de-DE.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/de-DE.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/en-US.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/en-US.ts
 /**
 * Copyright 2023-present DreamNum Co., Ltd.

package/lib/cjs/locale/es-ES.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/es-ES.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/fa-IR.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/fa-IR.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/fr-FR.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/fr-FR.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/id-ID.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/id-ID.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/it-IT.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/it-IT.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/ja-JP.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/ja-JP.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/ko-KR.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/ko-KR.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/pl-PL.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/pl-PL.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/pt-BR.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/pt-BR.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/ru-RU.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/ru-RU.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/sk-SK.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/sk-SK.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/vi-VN.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/vi-VN.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/zh-CN.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/zh-CN.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/zh-HK.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/zh-HK.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/cjs/locale/zh-TW.js

@@ -1,4 +1,3 @@
-
 //#region src/locale/zh-TW.ts
 const locale = { "sheets-data-validation": {
 	operators: {

package/lib/es/facade.js

@@ -1,12 +1,12 @@
 import { CanceledError, DataValidationErrorStyle, DataValidationOperator, DataValidationStatus, DataValidationType, ICommandService, IUniverInstanceService, generateRandomId } from "@univerjs/core";
-import { AddSheetDataValidationCommand, ClearRangeDataValidationCommand, RemoveSheetAllDataValidationCommand, RemoveSheetDataValidationCommand, SheetDataValidationModel, SheetsDataValidationValidatorService, UpdateSheetDataValidationOptionsCommand, UpdateSheetDataValidationRangeCommand, UpdateSheetDataValidationSettingCommand } from "@univerjs/sheets-data-validation";
+import { AddSheetDataValidationCommand, ClearRangeDataValidationCommand, RemoveSheetAllDataValidationCommand, RemoveSheetDataValidationCommand, SheetDataValidationModel, SheetsDataValidationValidatorService, UpdateSheetDataValidationOptionsCommand, UpdateSheetDataValidationRangeCommand, UpdateSheetDataValidationSettingCommand, getRangesInWorksheet } from "@univerjs/sheets-data-validation";
 import { FRange, FWorkbook, FWorksheet } from "@univerjs/sheets/facade";
 import { DataValidationModel, getRuleOptions } from "@univerjs/data-validation";
 import { serializeRangeToRefString } from "@univerjs/engine-formula";
 import { serializeListOptions } from "@univerjs/sheets";
 import { FEventName, FUniver } from "@univerjs/core/facade";
 
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/typeof.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/typeof.js
 function _typeof(o) {
 	"@babel/helpers - typeof";
 	return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function(o) {
@@ -17,7 +17,7 @@ function _typeof(o) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPrimitive.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPrimitive.js
 function toPrimitive(t, r) {
 	if ("object" != _typeof(t) || !t) return t;
 	var e = t[Symbol.toPrimitive];
@@ -30,14 +30,14 @@ function toPrimitive(t, r) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPropertyKey.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPropertyKey.js
 function toPropertyKey(t) {
 	var i = toPrimitive(t, "string");
 	return "symbol" == _typeof(i) ? i : i + "";
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/defineProperty.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/defineProperty.js
 function _defineProperty(e, r, t) {
 	return (r = toPropertyKey(r)) in e ? Object.defineProperty(e, r, {
 		value: t,
@@ -152,7 +152,7 @@ var FDataValidationBuilder = class FDataValidationBuilder {
 	* ```
 	*/
 	getAllowInvalid() {
-		return this._rule.errorStyle !== DataValidationErrorStyle.STOP;
+		return this._rule.showErrorMessage === false || this._rule.errorStyle !== DataValidationErrorStyle.STOP;
 	}
 	/**
 	* Gets the data validation type of the rule
@@ -960,7 +960,14 @@ var FDataValidationBuilder = class FDataValidationBuilder {
 */
 var FDataValidation = class {
 	constructor(rule, worksheet, _injector) {
-		_defineProperty(this, "rule", void 0);
+		_defineProperty(
+			this,
+			/**
+			* The underlying validation rule data. Use the Facade setters to update a rule attached to a worksheet.
+			*/
+			"rule",
+			void 0
+		);
 		_defineProperty(this, "_worksheet", void 0);
 		_defineProperty(this, "_injector", void 0);
 		this._injector = _injector;
@@ -982,7 +989,7 @@ var FDataValidation = class {
 	* ```
 	*/
 	getAllowInvalid() {
-		return this.rule.errorStyle !== DataValidationErrorStyle.STOP;
+		return this.rule.showErrorMessage === false || this.rule.errorStyle !== DataValidationErrorStyle.STOP;
 	}
 	/**
 	* Gets the data validation type of the rule
@@ -1128,8 +1135,8 @@ var FDataValidation = class {
 		return this.rule.ranges.map((range) => this._injector.createInstance(FRange, workbook, this._worksheet, range));
 	}
 	/**
-	* Gets the unit ID of the worksheet
-	* @returns {string | undefined} The unit ID of the worksheet
+	* Gets the unit ID of the workbook
+	* @returns {string | undefined} The unit ID of the workbook
 	* @example
 	* ```typescript
 	* const fWorkbook = univerAPI.getActiveWorkbook();
@@ -1359,7 +1366,7 @@ var FRangeSheetsDataValidationMixin = class extends FRange {
 		const worksheet = this._worksheet;
 		const sheetName = worksheet.getName();
 		const errors = [];
-		for (const range of ranges) {
+		for (const range of getRangesInWorksheet(ranges, worksheet)) {
 			const promises = [];
 			for (let row = range.startRow; row <= range.endRow; row++) for (let col = range.startColumn; col <= range.endColumn; col++) promises.push((async () => {
 				try {
@@ -1599,7 +1606,7 @@ var FWorkbookSheetsDataValidationMixin = class extends FWorkbook {
 		if (!worksheet) throw new Error(`Cannot find worksheet with sheetId: ${sheetId}`);
 		const sheetName = worksheet.getName();
 		const errors = [];
-		for (const range of ranges) {
+		for (const range of getRangesInWorksheet(ranges, worksheet)) {
 			const promises = [];
 			for (let row = range.startRow; row <= range.endRow; row++) for (let col = range.startColumn; col <= range.endColumn; col++) promises.push((async () => {
 				try {
@@ -1667,7 +1674,7 @@ var FWorksheetDataValidationMixin = class extends FWorksheet {
 		const worksheet = this._worksheet;
 		const sheetName = worksheet.getName();
 		const errors = [];
-		for (const range of ranges) {
+		for (const range of getRangesInWorksheet(ranges, worksheet)) {
 			const promises = [];
 			for (let row = range.startRow; row <= range.endRow; row++) for (let col = range.startColumn; col <= range.endColumn; col++) promises.push((async () => {
 				try {