Yes, for me it's a matter of style. Using with blocks might make the code easier to type but I find it harder to read once it gets cold. ... View more

I think I see what is wrong. You edited the path in a way that made it a complex path not seperate paths. Try this var thePathed = app.activeDocument; for (var a = 0; a < thePathed.pathItems.length; a++) {     app.activeDocument = thePathed;     var aPath = thePathed.pathItems;                for (var b = 0; b < aPath.subPathItems.length; b++)         {             if ( aPath.subPathItems.operation ==  ShapeOperation.SHAPEXOR || aPath.subPathItems.closed == false )             {                                            alert("Error in Path Name: "+ aPath.name)                                    }         }            } alert ("Path checking Finished") ... View more

Paul, Doesn't your code run the same action for both horz and vert docs. And assume they are 300 ppi? How about something like this #target photoshop var doc = app.activeDocument; var Width = doc.width.as('in'); var Height = doc.height.as('in'); var Max = Math.max(Width,Height); switch (Max){ case 6 : //6 inch longside                     if(activeDocument.width > activeDocument.height){                          doAction ("4x6_H", "actionSet");                     }else{                          doAction ("4x6_V", "actionSet");                     }                     break; case 7 :// 7 inch longside                     if(activeDocument.width > activeDocument.height){                          doAction ("5x7_H", "actionSet");                     }else{                          doAction ("5x7_V", "actionSet");                     }                     break; case 10 : // 10 inch longside                     if(activeDocument.width > activeDocument.height){                          doAction ("8x10_H", "actionSet");                     }else{                          doAction ("8x10_V", "actionSet");                     }                     break; default : break; } ... View more

It's not working for lots of reasons. Channels, and almost everything else start with 0. To use an index you have to use Channels. The Channel class does not have a copy method. And finally activeChannels expects an array. So it's easy fixes for everything except the copy method. For that we need to use scriptlistner to make an applyImage function. When you apply a image in normal mode it's the same as copy without going through the clipboard. // assumes grayscale mode with two alpha channels // make gray channel active app.activeDocument.activeChannels = [app.activeDocument.channels[0]]; // copy first alpha contents to gray applyChannelByName( app.activeDocument.channels[1].name ); // make first alpha channel active app.activeDocument.activeChannels = [app.activeDocument.channels[1]]; // copy second alpha content to frist alpha applyChannelByName( app.activeDocument.channels[2].name ); // remove the second alpha app.activeDocument.channels[2].remove(); // make gray channel active app.activeDocument.activeChannels = [app.activeDocument.channels[0]]; function applyChannelByName( name ) {     var desc = new ActionDescriptor();         var applyDesc = new ActionDescriptor();             var ref = new ActionReference();             ref.putName( charIDToTypeID('Chnl'), name);         applyDesc.putReference( charIDToTypeID('T   '), ref );         applyDesc.putBoolean( charIDToTypeID('PrsT'), true );     desc.putObject( charIDToTypeID('With'), charIDToTypeID('Clcl'), applyDesc );     executeAction( charIDToTypeID('AppI'), desc, DialogModes.NO ); } ... View more

If you just want to check 10px X 10px bottom right function hasBlackBottomRight(){      var doc = activeDocument;      var w = doc.width.as('px');      var h = doc.height.as('px');      var t = new Array( new UnitValue(w-10,'px'),new UnitValue(h-10,'px'));      var r = new Array( new UnitValue(w,'px'), new UnitValue(h-10,'px'));      var b = new Array( new UnitValue(w,'px'), new UnitValue(h,'px'));      var l = new Array( new UnitValue(w-10,'px'), new UnitValue(h,'px'));      doc.selection.select([t,r,b,l]);      return !!!doc.histogram.toString().match(/,[1-9]/g); } if( hasBlackBottomRight()){      // code to deal with black line at bottom;      alert('All black at bottom right'); }else{      // code to deal with missing black line;      alert('Non black colors at bottom right'); } ... View more