• Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 2.2
    • Component/s: None
    • Security Level: All
    • Labels:


      Pull-Request was automatically synchronized:

      Hi there,
      I'm wondering myself if the QueryBuilder::getQuery is a good method name.
      Actually, a getMethodObject makes me think that I can use many times the getMethodObject and work with the same object.
      e.g :

      $qb->getQuery()->useResultCache (true);
      In my code, the problem is that getQuery return a new instance of Query each time that I call getQuery.
      Furthermore, the getQuery method invoke a createQuery method.

      So, maybe the QueryBuiler::getQuery can store a reference to the created Query instance, and if the QueryBuilder::getQuery is call many times without any change return the stored instance.
      If the QueryBuilder::getQuery have to generate a new Query, throw an exception or something like that.

      Because of a change to QueryBuilder::createQuery is a major change, I don't think that It could be the solution, but it's a reflexion around this.

      What do you think about that ?


        beberlei Benjamin Eberlei created issue -
        guilhermeblanco Guilherme Blanco made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Benjamin Eberlei [ beberlei ] Guilherme Blanco [ guilhermeblanco ]
        Fix Version/s 2.2-DEV [ 10157 ]
        Resolution Won't Fix [ 2 ]
        beberlei Benjamin Eberlei made changes -
        Workflow jira [ 13262 ] jira-feedback [ 15147 ]
        beberlei Benjamin Eberlei made changes -
        Workflow jira-feedback [ 15147 ] jira-feedback2 [ 17011 ]
        beberlei Benjamin Eberlei made changes -
        Workflow jira-feedback2 [ 17011 ] jira-feedback3 [ 19264 ]


          • Assignee:
            guilhermeblanco Guilherme Blanco
            beberlei Benjamin Eberlei
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created: