Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Contest-2 Show index information in the data dict. #4678

Closed
pma-import opened this issue Mar 17, 2012 · 15 comments
Closed

Contest-2 Show index information in the data dict. #4678

pma-import opened this issue Mar 17, 2012 · 15 comments
Labels
Milestone

Comments

@pma-import
Copy link

attached is the patch.


@pma-import
Copy link
Author

  • assigned_to: nobody --> lem9

  • Original author: lem9

@pma-import
Copy link
Author

Thanks for the patch, but I cannot accept it.

There is code duplication with other parts of the code which already display the index information in other pages.


  • Original author: lem9

@pma-import
Copy link
Author

  • status: open --> pending-rejected

  • Original author: lem9

@pma-import
Copy link
Author

hey @lem9
I was sick and couldn't reply on my patch. I'm just back to my routine. I will reply about this patch soon.


  • Original author: hiddenpearls

@pma-import
Copy link
Author

@pma-import
Copy link
Author

Please have a look at code now. It is not duplicated.


  • Original author: hiddenpearls

@pma-import
Copy link
Author

  • status: pending-rejected --> open-rejected

  • Original author: lem9

@pma-import
Copy link
Author

This is better, but

  1. please provide a patch that applies to current master
  2. avoid showing the index section for a table that has no index

  • Original author: lem9

@pma-import
Copy link
Author

  • status: open-rejected --> open

  • Original author: lem9

@pma-import
Copy link
Author

  1. please provide a patch that applies to current master
    ok will do it

  2. avoid showing the index section for a table that has no index
    Currently it shows index section and displays that "No index defined!"
    We shouldn't show this ?


  • Original author: hiddenpearls

@pma-import
Copy link
Author

About #2, indeed it takes unnecessary room in the report.


  • Original author: lem9

@pma-import
Copy link
Author

From ee75ab459e86915b2a0dff794c8ad6b404c564b0 Mon Sep 17 00:00:00 2001
From: Muhammad Adnan <hiddenpearls@gmail.com>
Date: Thu, 22 Mar 2012 23:01:14 +0500
Subject: [PATCH] Contest-2: Show indexes information in data dictionary


db_datadict.php | 8 +++++++-
1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/db_datadict.php b/db_datadict.php
index e57206f..46f4b80 100644
--- a/db_datadict.php
+++ b/db_datadict.php
@@ -22,7 +22,7 @@ if (! isset($selected_tbl)) {
$cfgRelation = PMA_getRelationsParam();

require_once 'libraries/transformations.lib.php';

+require_once 'libraries/Index.class.php';

/**

  • Check parameters
    @@ -262,6 +262,12 @@ foreach ($tables as $table) {
    $count++;
    ?>
    </table>
    +<?php
    +// display indexes information
    • if(count(PMA_Index::getFromTable($table, $db))>0){
    •    echo PMA_Index::getView($table, $db,true);
      
    • }
      +?>
      </div>
      <?php
      } //ends main while
      --
      1.7.5.4

  • Original author: lem9

@pma-import
Copy link
Author

Your patch was merged in the repository, thanks.


  • Original author: lem9

@pma-import
Copy link
Author

  • summary: Contest-2 Show index information in the data dictionary --> (ok 4.0) Contest-2 Show index information in the data dict.
  • priority: 5 --> 1
  • status: open --> open-accepted

  • Original author: lem9

@pma-import
Copy link
Author

  • Status: open-accepted --> closed-fixed

  • Original author: nijel

@pma-import pma-import added this to the 4.0 milestone Jul 24, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants