Skip to:
Content
Pages
Categories
Search
Top
Bottom

Little sql bugs

  • Avatar of imath
    imath
    Moderator

    @imath

    Hi @slaFFik,

    Congratulations for this plugin, i just discovered it a few minutes ago !! Great job.

    I played a little with it and i think there’s a trouble in bp-gtm.php on line 166
    group_id bigint(20) NOT NULL_DEFAULT ‘0’,
    as a result the table is not created :(

    i corrected to this : group_id bigint(20) NOT NULL DEFAULT ‘0’, but then i had a new sql issue that i resolved adding the following code after global declaration to the function `bp_gtm_get_user_roles` in the file lib/roles.php :
    `if(!$bp->groups->current_group->id)
    return false;`

    Then i have a new bug in the function `bp_gtm_get_project_status` of the lib/projects.php. When creating a task, it seems that the project_id is not sent to the function, so i have a new sql error :(

    Hope you’ll find a fix to this last one ;)

    Finally, i have no involved user in `group-slug/gtm/involved`, i guess i should have here the user that are assigned to a task or a project…

    Again, very interesting plugin : Bravo ;)

Viewing 8 replies - 1 through 8 (of 8 total)
  • Avatar of imath
    imath
    Moderator

    @imath

    Ok i think i’ve found a fix to my last sql error (function `bp_gtm_get_project_status`)

    In bp-gtm-core.php around line 678 :
    replace this :
    `if ($_POST == 0) {
    $status = bp_gtm_get_project_status($project_id);
    $project_id = $_POST;
    }`
    by this :
    `if ($_POST == 0) {
    $project_id = $_POST;
    $status = bp_gtm_get_project_status($project_id);
    }`

    then as the function `bp_gtm_get_project_status` is called again at line 787, i added this code after line 781 :
    `if(!$project_id){
    $project_id = $_POST;
    }`
    (just in case)

    then i have the involved users in `group-slug/gtm/involved` !!

    all seems to work fine :)

    I love this plugin !!

    Avatar of josemv
    josemv
    Participant

    @el_terko

    Monsieur @imath

    Any Myhome widget for this plugin maybe ?

    Avatar of Slava UA
    Slava UA
    Participant

    @slaffik

    @imath
    Many thanks to you and your bug fixes! It’s rather difficult to find all bugs by myself only, that’s why I love BP community :)
    I will release updated version shortly (today/tomorrow).

    Avatar of imath
    imath
    Moderator

    @imath

    @slaFFik

    You’re welcome, i have the same difficulties to find my plugins bugs ;)

    Avatar of imath
    imath
    Moderator

    @imath

    @el_terko

    Interesting idea.
    I think BP GTM System already has a nice and full featured UI in members/username area.
    So far BP My Home only have widgets that don’t rely on another plugin, so i guess it’s a bit early to answer you…
    What’s sure is that i’m going to play a lot with BP GTM System !!

    Avatar of Slava UA
    Slava UA
    Participant

    @slaffik

    Thanks to you, @imath, v1.0.1 is out.
    And I will be happy if you write me what new features you would like to see in GTM System :)

    Avatar of imath
    imath
    Moderator

    @imath

    @slaFFik : great !!
    and i will ;)

    Avatar of AndreMartin
    AndreMartin
    Participant

    @andremartin

    Could this somehow be related to my error 404? It sounds familiar..

Viewing 8 replies - 1 through 8 (of 8 total)

You must be logged in to reply to this topic.