Fix for #76, editing upload que
Whew it took a bit but now the upload que on the page edit page is editable. You can add and remove files as needed without having to add all your files at once, which us how it previously worked. Still needs to be tested a bit but the plumbing is up and running. Also removed a stray php format config as it is no longer needed.pull/84/head
parent
a31dff94cb
commit
8734baf85e
@ -1,156 +0,0 @@
|
|||||||
<?xml version="1.0"?>
|
|
||||||
<ruleset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" name="PHP_CodeSniffer" xsi:noNamespaceSchemaLocation="phpcs.xsd">
|
|
||||||
<description>The coding standard for PHP_CodeSniffer itself.</description>
|
|
||||||
|
|
||||||
<file>autoload.php</file>
|
|
||||||
<file>bin</file>
|
|
||||||
<file>scripts</file>
|
|
||||||
<file>src</file>
|
|
||||||
<file>tests</file>
|
|
||||||
|
|
||||||
<exclude-pattern>*/src/Standards/*/Tests/*\.(inc|css|js)$</exclude-pattern>
|
|
||||||
<exclude-pattern>*/tests/Core/*/*\.(inc|css|js)$</exclude-pattern>
|
|
||||||
|
|
||||||
<arg name="basepath" value="."/>
|
|
||||||
<arg name="colors"/>
|
|
||||||
<arg name="parallel" value="75"/>
|
|
||||||
<arg value="np"/>
|
|
||||||
|
|
||||||
<!-- Don't hide tokenizer exceptions -->
|
|
||||||
<rule ref="Internal.Tokenizer.Exception">
|
|
||||||
<type>error</type>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Include the whole PEAR standard -->
|
|
||||||
<rule ref="PEAR">
|
|
||||||
<exclude name="PEAR.NamingConventions.ValidFunctionName"/>
|
|
||||||
<exclude name="PEAR.NamingConventions.ValidVariableName"/>
|
|
||||||
<exclude name="PEAR.Commenting.ClassComment"/>
|
|
||||||
<exclude name="PEAR.Commenting.FileComment.MissingCategoryTag"/>
|
|
||||||
<exclude name="PEAR.Commenting.FileComment.MissingPackageTag"/>
|
|
||||||
<exclude name="PEAR.Commenting.FileComment.MissingLinkTag"/>
|
|
||||||
<exclude name="PEAR.Commenting.FileComment.MissingVersion"/>
|
|
||||||
<exclude name="PEAR.Commenting.InlineComment"/>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Include some sniffs from other standards that don't conflict with PEAR -->
|
|
||||||
<rule ref="Squiz.Arrays.ArrayBracketSpacing"/>
|
|
||||||
<rule ref="Squiz.Arrays.ArrayDeclaration"/>
|
|
||||||
<rule ref="Squiz.Commenting.ClosingDeclarationComment"/>
|
|
||||||
<rule ref="Squiz.ControlStructures.ControlSignature"/>
|
|
||||||
<rule ref="Squiz.ControlStructures.ElseIfDeclaration"/>
|
|
||||||
<rule ref="Squiz.Commenting.BlockComment"/>
|
|
||||||
<rule ref="Squiz.Commenting.DocCommentAlignment"/>
|
|
||||||
<rule ref="Squiz.Commenting.EmptyCatchComment"/>
|
|
||||||
<rule ref="Squiz.Commenting.InlineComment"/>
|
|
||||||
<rule ref="Squiz.Commenting.LongConditionClosingComment"/>
|
|
||||||
<rule ref="Squiz.Commenting.PostStatementComment"/>
|
|
||||||
<rule ref="Squiz.Commenting.VariableComment"/>
|
|
||||||
<rule ref="Squiz.Formatting.OperatorBracket"/>
|
|
||||||
<rule ref="Squiz.Functions.FunctionDeclarationArgumentSpacing"/>
|
|
||||||
<rule ref="Squiz.Operators.ComparisonOperatorUsage"/>
|
|
||||||
<rule ref="Squiz.PHP.DisallowInlineIf"/>
|
|
||||||
<rule ref="Squiz.Scope.MethodScope"/>
|
|
||||||
<rule ref="Squiz.Strings.ConcatenationSpacing">
|
|
||||||
<severity>0</severity>
|
|
||||||
</rule>
|
|
||||||
<rule ref="Squiz.WhiteSpace.ControlStructureSpacing"/>
|
|
||||||
<rule ref="Squiz.WhiteSpace.FunctionClosingBraceSpace"/>
|
|
||||||
<rule ref="Squiz.WhiteSpace.FunctionSpacing"/>
|
|
||||||
<rule ref="Squiz.WhiteSpace.MemberVarSpacing"/>
|
|
||||||
<rule ref="Squiz.WhiteSpace.OperatorSpacing"/>
|
|
||||||
<rule ref="Squiz.WhiteSpace.SuperfluousWhitespace"/>
|
|
||||||
<rule ref="Generic.Arrays.DisallowLongArraySyntax"/>
|
|
||||||
<rule ref="Generic.Commenting.Todo"/>
|
|
||||||
<rule ref="Generic.ControlStructures.DisallowYodaConditions"/>
|
|
||||||
<rule ref="Generic.ControlStructures.InlineControlStructure"/>
|
|
||||||
<rule ref="Generic.Formatting.DisallowMultipleStatements"/>
|
|
||||||
<rule ref="Generic.Formatting.SpaceAfterCast"/>
|
|
||||||
<rule ref="Generic.NamingConventions.ConstructorName"/>
|
|
||||||
<rule ref="Generic.PHP.DeprecatedFunctions"/>
|
|
||||||
<rule ref="Generic.PHP.LowerCaseKeyword"/>
|
|
||||||
<rule ref="Generic.Strings.UnnecessaryStringConcat"/>
|
|
||||||
<rule ref="Generic.WhiteSpace.IncrementDecrementSpacing"/>
|
|
||||||
<rule ref="PSR2.Classes.PropertyDeclaration"/>
|
|
||||||
<rule ref="PSR2.Methods.MethodDeclaration"/>
|
|
||||||
<rule ref="PSR2.Files.EndFileNewline"/>
|
|
||||||
<rule ref="PSR12.Files.OpenTag"/>
|
|
||||||
<rule ref="Zend.Files.ClosingTag"/>
|
|
||||||
|
|
||||||
<!-- PEAR uses warnings for inline control structures, so switch back to errors -->
|
|
||||||
<rule ref="Generic.ControlStructures.InlineControlStructure">
|
|
||||||
<properties>
|
|
||||||
<property name="error" value="true"/>
|
|
||||||
</properties>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- We use custom indent rules for arrays -->
|
|
||||||
<rule ref="Generic.Arrays.ArrayIndent"/>
|
|
||||||
<rule ref="Squiz.Arrays.ArrayDeclaration.KeyNotAligned">
|
|
||||||
<severity>0</severity>
|
|
||||||
</rule>
|
|
||||||
<rule ref="Squiz.Arrays.ArrayDeclaration.ValueNotAligned">
|
|
||||||
<severity>0</severity>
|
|
||||||
</rule>
|
|
||||||
<rule ref="Squiz.Arrays.ArrayDeclaration.CloseBraceNotAligned">
|
|
||||||
<severity>0</severity>
|
|
||||||
</rule>
|
|
||||||
<rule ref="Squiz.Arrays.ArrayDeclaration.CloseBraceNewLine">
|
|
||||||
<severity>0</severity>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Check var names, but we don't want leading underscores for private vars -->
|
|
||||||
<rule ref="Squiz.NamingConventions.ValidVariableName"/>
|
|
||||||
<rule ref="Squiz.NamingConventions.ValidVariableName.PrivateNoUnderscore">
|
|
||||||
<severity>0</severity>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Only one argument per line in multi-line function calls -->
|
|
||||||
<rule ref="PEAR.Functions.FunctionCallSignature">
|
|
||||||
<properties>
|
|
||||||
<property name="allowMultipleArguments" value="false"/>
|
|
||||||
</properties>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Have 12 chars padding maximum and always show as errors -->
|
|
||||||
<rule ref="Generic.Formatting.MultipleStatementAlignment">
|
|
||||||
<properties>
|
|
||||||
<property name="maxPadding" value="12"/>
|
|
||||||
<property name="error" value="true"/>
|
|
||||||
</properties>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Ban some functions -->
|
|
||||||
<rule ref="Generic.PHP.ForbiddenFunctions">
|
|
||||||
<properties>
|
|
||||||
<property name="forbiddenFunctions" type="array">
|
|
||||||
<element key="sizeof" value="count"/>
|
|
||||||
<element key="delete" value="unset"/>
|
|
||||||
<element key="print" value="echo"/>
|
|
||||||
<element key="is_null" value="null"/>
|
|
||||||
<element key="create_function" value="null"/>
|
|
||||||
</property>
|
|
||||||
</properties>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Private methods MUST not be prefixed with an underscore -->
|
|
||||||
<rule ref="PSR2.Methods.MethodDeclaration.Underscore">
|
|
||||||
<type>error</type>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- Private properties MUST not be prefixed with an underscore -->
|
|
||||||
<rule ref="PSR2.Classes.PropertyDeclaration.Underscore">
|
|
||||||
<type>error</type>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- The testing bootstrap file uses string concats to stop IDEs seeing the class aliases -->
|
|
||||||
<rule ref="Generic.Strings.UnnecessaryStringConcat">
|
|
||||||
<exclude-pattern>tests/bootstrap\.php</exclude-pattern>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
<!-- This test file specifically *needs* Windows line endings for testing purposes. -->
|
|
||||||
<rule ref="Generic.Files.LineEndings.InvalidEOLChar">
|
|
||||||
<exclude-pattern>tests/Core/Tokenizer/StableCommentWhitespaceWinTest\.php</exclude-pattern>
|
|
||||||
</rule>
|
|
||||||
|
|
||||||
</ruleset>
|
|
Loading…
Reference in New Issue