Get rid of zope.app.zapi

Bug #219302 reported by Lorenzo Gil Sanchez on 2008-04-18
4
Affects Status Importance Assigned to Milestone
Zope 3
Undecided
Unassigned

Bug Description

AFAIK current coding style practices in Zope3 are against the use of zope.app.zapi package. Instead it is recommended to use the packages that zope.app.zapi wrapps, directly.

The purpose of this bug report is to provide a place where I can attach patches (since I'm not a commiter) for every zope.app.* package that use zope.app.zapi so we can get rid of it.

If you think adding so many patches to one bug report is a little bit overkill, please let me know another way of doing so and I'll be happy to do it.

Lorenzo Gil Sanchez (lgs) wrote :

On 18 Apr 2008, at 19:32 , Lorenzo Gil Sanchez wrote:
> Public bug reported:
>
> AFAIK current coding style practices in Zope3 are against the use of
> zope.app.zapi package. Instead it is recommended to use the packages
> that zope.app.zapi wrapps, directly.
>
> The purpose of this bug report is to provide a place where I can
> attach
> patches (since I'm not a commiter) for every zope.app.* package that
> use
> zope.app.zapi so we can get rid of it.
>
> If you think adding so many patches to one bug report is a little bit
> overkill, please let me know another way of doing so and I'll be happy
> to do it.

Can't you just become a commiter? :)

Lorenzo Gil Sanchez (lgs) wrote :

Well, I think I may lack some confidence for touching svn.zope.org. That's why I thought about this simple-easy-project to get used to zope3 development standards and procedures.

On the other hand I already have 18 patches and many more will come. It may be a non trivial task for a commiter to apply them all but for the person who is writting them (me) it can be pretty simple to just commit them.

On 18 Apr 2008, at 20:06 , Lorenzo Gil Sanchez wrote:
> Well, I think I may lack some confidence for touching svn.zope.org.
> That's why I thought about this simple-easy-project to get used to
> zope3
> development standards and procedures.
>
> On the other hand I already have 18 patches and many more will come.
> It
> may be a non trivial task for a commiter to apply them all but for the
> person who is writting them (me) it can be pretty simple to just
> commit
> them.

Exactly :). Please become a committer.

Lorenzo Gil Sanchez (lgs) wrote :

Ok, I won't attach more patches to this bug. Instead I will apply for a svn account and apply the patches myself.

I'll leave the bug open until we get rid of zope.app.zapi

On 18 Apr 2008, at 20:54 , Lorenzo Gil Sanchez wrote:
> Ok, I won't attach more patches to this bug. Instead I will apply
> for a
> svn account and apply the patches myself.
>
> I'll leave the bug open until we get rid of zope.app.zapi

While we can get rid of the imports of zope.app.zapi, we can't get rid
of the zope.app.zapi package for backwards compatibility reasons.
Let's just leave it out there to bit rot and make sure that no other
zope.* or zope.app.* package depends on it.

Lorenzo Gil Sanchez (lgs) wrote :

I've removed all existence of zope.app.zapi form the zope.* and zope.app.* namespaces so I'll close the bug now.

Changed in zope3:
status: New → Fix Committed
Dan Korostelev (nadako) wrote :

The packages without zope.app.zapi were released.

Changed in zope3:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers