•  
      request #8972 Get rid of all direct calls to mysql API
    Infos
    #8972
    Thomas Gerbet (tgerbet)
    2017-12-29 09:27
    2016-03-19 00:47
    9076
    Details
    Get rid of all direct calls to mysql API
    Tuleap exposes ways to communicate with a MySQL database. Direct calls to the mysql API should be avoided as it makes harder future modification on how Tuleap exchanges with a database.

    There are multiples legacy pieces of code that still use the mysql API directly.

    Done
    ====
    [x] LDAP plugin (gerrit #5292)
    [x] Docman plugin (gerrit #5290)
    [x] ForumML plugin (gerrit #5291)
    [x] LDAP plugin (gerrit #5292)
    [x] project_export_utils.php (gerrit #5294)
    [x] ProjectCreator::createGroupEntry (gerrit #8450)
    [x] TV3 (dump of attachments) (gerrit #10230)
    [x] Mediawiki plugin (lost databases identifier script) (gerrit #10231)


    Out of scope for this request
    =======================
    - PHPWiki
    Empty
    All
    Empty
    • [ ] enhancement
    • [x] internal improvement
    Empty
    Stage
    Empty
    Closed
    2017-12-29
    Attachments
    Empty
    References
    Referencing request #8972

    Artifact Tracker v5

    rel #8982 8.15

    Git commit

    tuleap/tuleap/stable

    Merge commit 'refs/changes/92/5292/1' of ssh://gerrit.tuleap.net:29418/tuleap into stable 14da854636
    request #8972: LDAP plugin does not directly use mysql_* API anymore fa5b2cde81
    Merge commit 'refs/changes/91/5291/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD ad4d468d24
    request #8972: ForumML plugin does not directly use mysql_* API anymore 19a9b8226f
    Merge commit 'refs/changes/90/5290/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD 8a04d299e8
    request #8972: Docman plugin does not directly use mysql_* API anymore 4b127a2162
    Merge commit 'refs/changes/94/5294/1' of ssh://gerrit.tuleap.net:29418/tuleap into stable c0bba1a12c
    request #8972: Utilities to export project does not directly use mysql_* API anymore 574a1b83bf
    Merge commit 'refs/changes/50/8450/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD cd850f473b
    request #8972: ProjectCreator does not directly use mysql_* API anymore 143dd945ba
    Merge commit 'refs/changes/30/10230/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD 6bf3b95e1d
    request #8972: Util script to dump TV3 attachment does not directly use mysql_* API anymore 7054f809bd
    Merge commit 'refs/changes/31/10231/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD f47c056bdf
    request #8972: Mediawiki plugin does not directly use mysql_* API anymore 7fe92f254f

    Follow-ups

    User avatar
    Thomas Gerbet (tgerbet)2017-12-29 09:27
    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    • Status changed from Under review to Closed
    • Connected artifacts
    • Close date set to 2017-12-29
    User avatar
    Thomas Gerbet (tgerbet)2017-12-28 18:24
    Last 2 reviews for this request are finally here, see gerrit #10230 and gerrit #10231.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    • Status changed from Under implementation to Under review
    User avatar
    Thomas Gerbet (tgerbet)2017-12-28 18:03
    PHPWiki still rely on the mysql_* API but it's isolated from the rest of Tuleap and it's needing dedicated work to make it works properly with something else, this is out of scope for this request.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2017-12-28 17:58
    Abandonning the contribution to the IM plugin. IM plugin is deprecated and should be removed. gerrit #5393 has been abandoned.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    gerrit #8450 integrated into Tuleap 9.7.99.129

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2017-05-19 22:11
    See gerrit #8450 for ProjectCreator.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2017-04-20 22:09
    Update the remaining work with the latest data.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    gerrit #5294 (project_export_utils) integrated into Tuleap 8.14.99.51

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    gerrit #5292 for LDAP plugin integrated

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2016-03-19 01:44
    See gerrit #5294 for project_export_utils.php.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2016-03-19 01:12
    See gerrit #5293 for IM plugin.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2016-03-19 01:02
    See gerrit #5292 for LDAP plugin.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2016-03-19 00:56
    See gerrit #5291 for ForumML plugin.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes
    User avatar
    Thomas Gerbet (tgerbet)2016-03-19 00:51
    See gerrit #5290 for Docman plugin.

    • Original Submission
      Something went wrong, the follow up content couldn't be loaded
      Only formatting have been changed, you should switch to markup to see the changes