The basePermission parameters leads to a few things: * being used in ofbiz-component.xml of the new component * being used in the *SecurityPermissionSeedData.xml of the new component * being used in the *SecurityGroupDemoData.xml of the new component
A specific security group is not set in the ofbiz-component.xml, where OFBTOOLS is also defined. For what it is worth, we (in our environment) differentiate between componentID and componentName and have enhanced the ANT task also in such a way that we can make new components tenant specific and/or component type specific. Best regards, Pierre Smits ORRTIZ.COM <http://www.orrtiz.com> OFBiz based solutions & services OFBiz Extensions Marketplace http://oem.ofbizci.net/oci-2/ On Tue, Jul 19, 2016 at 11:47 AM, Taher Alkhateeb < slidingfilame...@gmail.com> wrote: > I know a little fast, but I have a patch that does the following: > > componentName: mandatory > componentResourceName: optional, defaults to componentName > webappName: optional, defaults to componentName > basePermission: optional, defaults to ${componentName}_ADMIN > > so the security permissions are OFBTOOLS followed by ${componentName}_ADMIN > > Total size of new code? 9 lines :) > > Whenever you folks want to I can push this to whatever JIRA requires it. > > On Tue, Jul 19, 2016 at 12:33 PM, Pierre Smits <pierre.sm...@gmail.com> > wrote: > > > Taher, > > > > What you are saying is not off topic. Thus it is good to bring these to > the > > table. And they need to be addressed before a solution can gather > consensus > > and gets implemented. > > > > Best regards, > > > > Pierre Smits > > > > ORRTIZ.COM <http://www.orrtiz.com> > > OFBiz based solutions & services > > > > OFBiz Extensions Marketplace > > http://oem.ofbizci.net/oci-2/ > > > > On Tue, Jul 19, 2016 at 11:27 AM, Taher Alkhateeb < > > slidingfilame...@gmail.com> wrote: > > > > > By the way, this is a good example of the problems of interactive > tasks. > > If > > > we create an interactive task for createComponent then we have to > update > > > its implementation because it will break based on these new > requirements. > > > > > > So everytime we change something on which an interactive task depends, > we > > > have to update that task as well. Usually interactive tasks have more > > > business logic for validation and hence are more brittle to such > > changes. I > > > know this is off topic but couldn't help notice it. > > > > > > On Tue, Jul 19, 2016 at 12:22 PM, Jacopo Cappellato < > > > jacopo.cappell...@hotwaxsystems.com> wrote: > > > > > > > +1 to the idea, with defaults for optional resources (1 mandatory, 3 > > > > optional). > > > > > > > > Wish: it would be nice to have more control on the component > structure; > > > for > > > > example create a component without a webapp, add a webapp to an > > existing > > > > component etc... > > > > > > > > Jacopo > > > > > > > > On Tue, Jul 19, 2016 at 11:15 AM, Taher Alkhateeb < > > > > slidingfilame...@gmail.com> wrote: > > > > > > > > > Noted, thank you. > > > > > > > > > > On Tue, Jul 19, 2016 at 12:09 PM, Pierre Smits < > > pierre.sm...@gmail.com > > > > > > > > > wrote: > > > > > > > > > > > Hi Taher, > > > > > > > > > > > > OFBTOOLS is not required as it is already defined in the template > > to > > > be > > > > > > included. When added in basePermission it leads to duplication in > > the > > > > > > generated ofbiz-component.xml file. > > > > > > > > > > > > I already created an issue for that, see > > > > > > https://issues.apache.org/jira/browse/OFBIZ-7785 > > > > > > > > > > > > Best regards, > > > > > > > > > > > > Pierre Smits > > > > > > > > > > > > ORRTIZ.COM <http://www.orrtiz.com> > > > > > > OFBiz based solutions & services > > > > > > > > > > > > OFBiz Extensions Marketplace > > > > > > http://oem.ofbizci.net/oci-2/ > > > > > > > > > > > > On Tue, Jul 19, 2016 at 11:04 AM, Taher Alkhateeb < > > > > > > slidingfilame...@gmail.com> wrote: > > > > > > > > > > > > > Hi Pierre, > > > > > > > > > > > > > > Hmmm, that's actually a nice idea. I suggest however we avoid > > > > > componentId > > > > > > > and just apply the following to the variables: > > > > > > > > > > > > > > componentName: mandatory > > > > > > > componentResourceName: optional, defaults to componentName > > > > > > > webappName: optional, defaults to componentName > > > > > > > basePermission: optional, defaults to OFBTOOLS (or whatever > > > defaults > > > > > > were) > > > > > > > > > > > > > > This way, You get a real nice and simple command for end-users > to > > > > > create > > > > > > > new components while at the same time allowing more > > expressiveness. > > > > > This > > > > > > > becomes similar to the createTenant task which also has > > everything > > > > > > optional > > > > > > > except the tenantId and tenantName > > > > > > > > > > > > > > Taher Alkhateeb > > > > > > > > > > > > > > On Tue, Jul 19, 2016 at 11:58 AM, Pierre Smits < > > > > pierre.sm...@gmail.com > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > Hi all, > > > > > > > > > > > > > > > > Currently our create component process (./gradlew > > > createComponent) > > > > is > > > > > > > error > > > > > > > > prone. > > > > > > > > > > > > > > > > It requires 4 variables to successfully create a new > component > > in > > > > the > > > > > > > > hot-deploy folder to jumpstart development with a component > > > > skeleton. > > > > > > > These > > > > > > > > variables are: > > > > > > > > > > > > > > > > - componentName > > > > > > > > - componentResourceName > > > > > > > > - webappName > > > > > > > > - basePermission > > > > > > > > > > > > > > > > I believe we should reduce the number of variables to just 1 > > > > variable > > > > > > > (e.g. > > > > > > > > componentId) to create consistency throughout the skeleton > and > > > its > > > > > > > > associated datasets. The developer can, after a skeleton has > > been > > > > > > built, > > > > > > > > then apply any kind of changes according to his/her > > requirements > > > an > > > > > > > needs. > > > > > > > > > > > > > > > > What is your opinion on this? > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > Pierre Smits > > > > > > > > > > > > > > > > ORRTIZ.COM <http://www.orrtiz.com> > > > > > > > > OFBiz based solutions & services > > > > > > > > > > > > > > > > OFBiz Extensions Marketplace > > > > > > > > http://oem.ofbizci.net/oci-2/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >