[Flow] TYPO3.TYPO3CR depends on TYPO3.Neos

Sat Dec 21 19:50:44 CET 2013

I'll add it again - after all it does make good sense to be able to count
unpublished nodes :-)

Have a great vacation :-)

Søren

On Sat, Dec 21, 2013 at 1:02 AM, Steffen Wickham <steffen at gaming-inc.de>wrote:

> I took some time to think about it and I think it is better to keep the
> getUnpublishedNodesCount method. I will not use it right now but there will
> be someone who need it for sure.
>
> As you said, there will be some cases where it is useful. One major
> "client" is Neos.... Yeah Neos can put it in its own PublishService but it
> should be a core feature to provide this feature.
>
> Would you be so kind to put it back in the patch and create a new patch
> set please? I'm on my way to my family for vacation and only got a bugy
> windows netbook for the next days ;-)
>
> Cheers
> Steffen
>
> Am 20.12.2013 10:52 schrieb =?ISO-8859-1?Q?S=F8ren_Malling?= <
> soren.malling at gmail.com>:
> >
> > Hmm, maybe it makes sense to have the getUnpublishedNodesCount..
> >
> > ping me if you see it as general usefull and we'll add it - I can imagine
> > some cases where it will be useful :-)
> >
> > Cheers
> >
> > Søren
> >
> >
> > On Fri, Dec 20, 2013 at 10:44 AM, Steffen Wickham <steffen at gaming-inc.de
> >wrote:
> >
> > > Good morning everybody! :)
> > >
> > > Thanks for the new patch set and the contribution article, Søren. Will
> > > have a look on the article later ;-)
> > >
> > > I'm fine with removing the getUnpublishedNodesCount as it can be
> derived
> > > from the main function call.
> > > I can't imagine where I would use it directly in my application so i
> > > approved the patch set.
> > >
> > > greetings
> > > Steffen
> > >
> > >
> > > Am 20.12.13 10:01, schrieb Søren Malling:
> > > > Nevermind, I did a new patchset :-)
> > > >
> > > > I removed getUnpublishedNodesCount from the interface - would you
> > > consider
> > > > it a usefull function in your work with TYPO3.TYPO3CR ? Because then
> > > we'll
> > > > introduce it :-)
> > > >
> > > > Cheers
> > > >
> > > > Søren
> > > >
> > > >
> > > > On Fri, Dec 20, 2013 at 9:35 AM, Søren Malling <
> soren.malling at gmail.com
> > > >wrote:
> > > >
> > > >> Hi Steffen,
> > > >>
> > > >> Please comment inside the review so we can keep track of people
> comments
> > > >> for the different patchsets :-)
> > > >>
> > > >> Regarding contribution, have a look at this walkthrough
> > > >>
> > > >> http://wiki.typo3.org/Contribution_Walkthrough_with_CommandLine
> > > >>
> > > >> I will update the patches later today if you don't do it before me
> > > >> (challenge?)  ;-)
> > > >>
> > > >> Cheers
> > > >>
> > > >> Søren
> > > >>
> > > >>
> > > >> On Thu, Dec 19, 2013 at 10:00 PM, Steffen Wickham <
> > > steffen at gaming-inc.de
> > > >>> wrote:
> > > >>> Hello Søren,
> > > >>>
> > > >>> thank you for your quick response! :)
> > > >>> I found two issues in the patch set fot TYPO3CR.
> > > >>>
> > > >>> First of all the required method "getUnpublishedNodesCount()"
> defined
> > > in
> > > >>> the PublishServiceInterface wasn't included in the PublishService.
> I
> > > >>> took the snipped from the same Neos method you got the other stuff
> so
> > > >>> maybe it has to be removed in neos as well.
> > > >>> The second issue were missing references to the Workspace and
> > > >>> NodeInterface Model within the interface definition so I'd added
> them
> > > >>> myself.
> > > >>>
> > > >>> I'm sorry that I didn't upload the patches by myself to Gerrit but
> I'm
> > > >>> not able to handle gerrit by my own :-(
> > > >>> So please have a look at my diff to the patch set 2 from gerrit at
> > > >>> https://gist.github.com/anonymous/8046156
> > > >>>
> > > >>> Cheers
> > > >>> Steffen
> > > >>>
> > > >>>
> > > >>>
> > > >>> Am 19.12.2013 17:08, schrieb Søren Malling:
> > > >>>> Thanks for bringing up again, I've been wanting to fix this for
> quiet
> > > >>> some
> > > >>>> time!
> > > >>>>
> > > >>>> Please test the following two patches (or at least the one for
> > > >>>> TYPO3.TYPO3CR if you don't work with Neos)
> > > >>>>
> > > >>>> For TYPO3.TYPO3CR
> > > >>>> https://review.typo3.org/#/c/26495/
> > > >>>>
> > > >>>> For TYPO3.Neos
> > > >>>> https://review.typo3.org/#/c/26496/
> > > >>>>
> > > >>>> Find it here
> > > >>>>
> > > >>>>
> > > https://review.typo3.org/#/q/status:open+branch:master+topic:46464,n,z
> > > >>>>
> > > >>>> Cheers
> > > >>>>
> > > >>>> Søren
> > > >>>>
> > > >>>>
> > > >>>>
> > > >>>> On Thu, Dec 19, 2013 at 4:34 PM, Steffen Wickham <
> > > >> steffen at gaming-inc.de
> > > >>>> wrote:
> > > >>>>
> > > >>>>> Hey guys,
> > > >>>>>
> > > >>>>> I'm trying to play around with TYPO3.TYPO3CR but got in trouble
> as
> > > >> there
> > > >>>>> is a hard coded linkage to "TYPO3.Neos", which I won't install
> for
> > > >> this
> > > >>>>> project.
> > > >>>>>
> > > >>>>> In \TYPO3\TYPO3CR\Domain\Model\Workspace there is an injection
> for
> > > the
> > > >>>>> Neos "publishingService" (lines 68-72) which will be used in the
> > > >> method
> > > >>>>> "publish()" (lines 130-142) only. I don't know if it is really
> > > >> necessary
> > > >>>>> to put in this linkage but it will break dependecies as Neos is
> not a
> > > >>>>> required package for composer.
> > > >>>>>
> > > >>>>> So I will suggest to review this code part.
> > > >>>>>
> > > >>>>> Have a nice day! :)
> > > >>>>> Steffen
> > > >>>>> _______________________________________________
> > > >>>>> Flow mailing list
> > > >>>>> Flow at lists.typo3.org
> > > >>>>> http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> > > >>>>>
> > > >>>> _______________________________________________
> > > >>>> Flow mailing list
> > > >>>> Flow at lists.typo3.org
> > > >>>> http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> > > >>> _______________________________________________
> > > >>> Flow mailing list
> > > >>> Flow at lists.typo3.org
> > > >>> http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> > > >>>
> > > >> _______________________________________________
> > > >> Flow mailing list
> > > >> Flow at lists.typo3.org
> > > >> http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> > > >>
> > > > _______________________________________________
> > > > Flow mailing list
> > > > Flow at lists.typo3.org
> > > > http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> > >
> > > _______________________________________________
> > > Flow mailing list
> > > Flow at lists.typo3.org
> > > http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> > >
> > _______________________________________________
> > Flow mailing list
> > Flow at lists.typo3.org
> > http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
> _______________________________________________
> Flow mailing list
> Flow at lists.typo3.org
> http://lists.typo3.org/cgi-bin/mailman/listinfo/flow
>