Skip to content

Commit

Permalink
batch → bulk
Browse files Browse the repository at this point in the history
  • Loading branch information
brandonkelly committed Dec 7, 2022
1 parent 39d76dc commit 615c267
Show file tree
Hide file tree
Showing 17 changed files with 36 additions and 18 deletions.
2 changes: 1 addition & 1 deletion src/elements/actions/CopyReferenceTag.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
activate: function(\$selectedItems)
{
Craft.ui.createCopyTextPrompt({
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/CopyUrl.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
return !!\$selectedItems.find('.element').data('url');
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/DeleteUsers.php
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: true,
bulk: true,
validateSelection: function(\$selectedItems)
{
for (var i = 0; i < \$selectedItems.length; i++)
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/Edit.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
return Garnish.hasAttr(\$selectedItems.find('.element'), 'data-editable');
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/EditImage.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
_imageEditor: null,
validateSelection: function(\$selectedItems)
{
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/NewChild.php
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ public function getTriggerHtml()
(() => {
let trigger = new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
return (!$maxLevels || $maxLevels > \$selectedItems.find('.element').data('level'));
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/NewSiblingAfter.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public function getTriggerHtml()
(() => {
let trigger = new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
activate: function(\$selectedItems)
{
Craft.redirectTo(Craft.getUrl($newSiblingUrl, 'after='+\$selectedItems.find('.element').data('id')));
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/NewSiblingBefore.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public function getTriggerHtml()
(() => {
let trigger = new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
activate: function(\$selectedItems)
{
Craft.redirectTo(Craft.getUrl($newSiblingUrl, 'before='+\$selectedItems.find('.element').data('id')));
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/PreviewAsset.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
return \$selectedItems.length === 1;
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/RenameFile.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
return Garnish.hasAttr(\$selectedItems.find('.element'), 'data-movable');
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/ReplaceFile.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
return Garnish.hasAttr(\$selectedItems.find('.element'), 'data-replaceable');
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/SuspendUsers.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: $type,
batch: true,
bulk: true,
validateSelection: \$selectedItems => {
for (let i = 0; i < \$selectedItems.length; i++) {
const \$element = \$selectedItems.eq(i).find('.element');
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/UnsuspendUsers.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: $type,
batch: true,
bulk: true,
validateSelection: \$selectedItems => {
for (let i = 0; i < \$selectedItems.length; i++) {
const \$element = \$selectedItems.eq(i).find('.element');
Expand Down
2 changes: 1 addition & 1 deletion src/elements/actions/View.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public function getTriggerHtml()
(() => {
new Craft.ElementActionTrigger({
type: {$type},
batch: false,
bulk: false,
validateSelection: function(\$selectedItems)
{
var \$element = \$selectedItems.find('.element');
Expand Down
2 changes: 1 addition & 1 deletion src/web/assets/cp/dist/cp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/web/assets/cp/dist/cp.js.map

Large diffs are not rendered by default.

22 changes: 20 additions & 2 deletions src/web/assets/cp/src/js/ElementActionTrigger.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,24 @@ Craft.ElementActionTrigger = Garnish.Base.extend(
triggerEnabled: true,

init: function (settings) {
if (!$.isPlainObject(settings)) {
settings = {};
}

// batch => bulk
if (typeof settings.batch !== 'undefined') {
settings.bulk = settings.batch;
delete settings.batch;
}
Object.defineProperty(settings, 'batch', {
get() {
return this.bulk;
},
set(v) {
this.bulk = v;
},
});

this.setSettings(settings, Craft.ElementActionTrigger.defaults);

this.$trigger = $(
Expand Down Expand Up @@ -57,7 +75,7 @@ Craft.ElementActionTrigger = Garnish.Base.extend(
var valid = true;
this.$selectedItems = Craft.elementIndex.getSelectedElements();

if (!this.settings.batch && this.$selectedItems.length > 1) {
if (!this.settings.bulk && this.$selectedItems.length > 1) {
valid = false;
} else if (typeof this.settings.validateSelection === 'function') {
valid = this.settings.validateSelection(this.$selectedItems);
Expand Down Expand Up @@ -96,7 +114,7 @@ Craft.ElementActionTrigger = Garnish.Base.extend(
{
defaults: {
type: null,
batch: true,
bulk: true,
validateSelection: null,
activate: null,
},
Expand Down

0 comments on commit 615c267

Please sign in to comment.