#4 Dynamic Content Insertion

closed-fixed
None
5
2008-02-25
2008-02-18
Ahmed Ashour
No

Dear all,

Please find patch for correct handling of pushInputSource().

Many thanks.

Discussion

  • Ahmed Ashour
    Ahmed Ashour
    2008-02-18

    Patch

     
    Attachments
  • Ahmed Ashour
    Ahmed Ashour
    2008-02-24

    Second Patch

     
  • Ahmed Ashour
    Ahmed Ashour
    2008-02-24

    Logged In: YES
    user_id=950730
    Originator: YES

    Second patch that has another method 'evaluateInputSource()', which can be removed to anything more appropriate. This patch doesn't break existing functionalities.

    Hope you find this useful.
    File Added: pushInputSource2.patch

     
  • Ahmed Ashour
    Ahmed Ashour
    2008-02-24

    Logged In: YES
    user_id=950730
    Originator: YES

    "... which can be removed to anything ..."

    I meant: which can be renamed to anything else.

     
  • Marc Guillemot
    Marc Guillemot
    2008-02-25

    Logged In: YES
    user_id=402164
    Originator: NO

    patch committed. Thanks.

    It is possible that the name of the method gets changed before next release.

     
  • Marc Guillemot
    Marc Guillemot
    2008-02-25

    • assigned_to: nobody --> mguillem
    • status: open --> closed-fixed