You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Jacopo Cappellato <ja...@hotwaxmedia.com> on 2012/06/22 06:46:45 UTC
Re: svn commit: r1352769 - in /ofbiz/trunk/applications/accounting: data/AccountingSecurityGroupDemoData.xml data/AccountingSecurityPermissionSeedData.xml widget/AccountingMenus.xml
Hans,
I did only a cursory review but I don't understand the reason for the removal of the rntries below... was it intentional?
Regards,
Jacopo
On Jun 22, 2012, at 6:12 AM, hansbak@apache.org wrote:
> Modified: ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml?rev=1352769&r1=1352768&r2=1352769&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml (original)
> +++ ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml Fri Jun 22 04:12:11 2012
> @@ -19,7 +19,6 @@ under the License.
> -->
> <entity-engine-xml>
> <!-- Payment Processors security -->
> - <SecurityGroupPermission groupId="FULLADMIN" permissionId="PAYPROC_ADMIN"/>
> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="PAYPROC_CREATE"/>
> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="PAYPROC_DELETE"/>
> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="PAYPROC_VIEW"/>
> @@ -45,7 +44,6 @@ under the License.
> <SecurityGroupPermission groupId="BIZADMIN" permissionId="ACCOUNTING_ADMIN"/>
>
> <!-- Manual Payment security -->
> - <SecurityGroupPermission groupId="FULLADMIN" permissionId="MANUAL_PAYMENT"/>
> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="MANUAL_PAYMENT"/>
> <SecurityGroupPermission groupId="BIZADMIN" permissionId="MANUAL_PAYMENT"/>
>
>
> Modified: ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml?rev=1352769&r1=1352768&r2=1352769&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml (original)
> +++ ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml Fri Jun 22 04:12:11 2012
> @@ -68,7 +68,6 @@ under the License.
>
> <!-- add admin to SUPER permission group -->
> <SecurityGroupPermission groupId="SUPER" permissionId="ACCOUNTING_ADMIN"/>
> - <SecurityGroupPermission groupId="SUPER" permissionId="PAYPROC_ADMIN"/>
> <SecurityGroupPermission groupId="SUPER" permissionId="PAY_INFO_ADMIN"/>
> <SecurityGroupPermission groupId="SUPER" permissionId="ACCOUNTING_COMM_VIEW"/>
> <SecurityGroupPermission groupId="SUPER" permissionId="ACCOUNTING_PRINT_CHECKS"/>
Re: svn commit: r1352769 - in /ofbiz/trunk/applications/accounting:
data/AccountingSecurityGroupDemoData.xml data/AccountingSecurityPermissionSeedData.xml
widget/AccountingMenus.xml
Posted by Hans Bakker <ma...@antwebsystems.com>.
Yes these were intentional because they are now covered by the
account_admin permission.
On 06/22/2012 11:46 AM, Jacopo Cappellato wrote:
> Hans,
>
> I did only a cursory review but I don't understand the reason for the removal of the rntries below... was it intentional?
>
> Regards,
>
> Jacopo
>
> On Jun 22, 2012, at 6:12 AM, hansbak@apache.org wrote:
>
>> Modified: ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml?rev=1352769&r1=1352768&r2=1352769&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml (original)
>> +++ ofbiz/trunk/applications/accounting/data/AccountingSecurityGroupDemoData.xml Fri Jun 22 04:12:11 2012
>> @@ -19,7 +19,6 @@ under the License.
>> -->
>> <entity-engine-xml>
>> <!-- Payment Processors security -->
>> -<SecurityGroupPermission groupId="FULLADMIN" permissionId="PAYPROC_ADMIN"/>
>> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="PAYPROC_CREATE"/>
>> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="PAYPROC_DELETE"/>
>> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="PAYPROC_VIEW"/>
>> @@ -45,7 +44,6 @@ under the License.
>> <SecurityGroupPermission groupId="BIZADMIN" permissionId="ACCOUNTING_ADMIN"/>
>>
>> <!-- Manual Payment security -->
>> -<SecurityGroupPermission groupId="FULLADMIN" permissionId="MANUAL_PAYMENT"/>
>> <SecurityGroupPermission groupId="FLEXADMIN" permissionId="MANUAL_PAYMENT"/>
>> <SecurityGroupPermission groupId="BIZADMIN" permissionId="MANUAL_PAYMENT"/>
>>
>>
>> Modified: ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml?rev=1352769&r1=1352768&r2=1352769&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml (original)
>> +++ ofbiz/trunk/applications/accounting/data/AccountingSecurityPermissionSeedData.xml Fri Jun 22 04:12:11 2012
>> @@ -68,7 +68,6 @@ under the License.
>>
>> <!-- add admin to SUPER permission group -->
>> <SecurityGroupPermission groupId="SUPER" permissionId="ACCOUNTING_ADMIN"/>
>> -<SecurityGroupPermission groupId="SUPER" permissionId="PAYPROC_ADMIN"/>
>> <SecurityGroupPermission groupId="SUPER" permissionId="PAY_INFO_ADMIN"/>
>> <SecurityGroupPermission groupId="SUPER" permissionId="ACCOUNTING_COMM_VIEW"/>
>> <SecurityGroupPermission groupId="SUPER" permissionId="ACCOUNTING_PRINT_CHECKS"/>