Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #118 from sudwebdesign/master
5711: Fixes mysql_query to dbquery && fatal error on save social prefs. Thanks for that sudwebdesign! If you register on our issue tracker, I can assign the fix to you. 
http://www.atutor.ca/atutor/mantis/
  • Loading branch information
ATutor committed Aug 24, 2016
2 parents a511644 + e9cac81 commit 111a69d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Expand Up @@ -134,7 +134,7 @@ function updatePrivacyPreference($member_id, $prefs){
global $db, $addslashes;

$member_id = intval($member_id);
$prefs = $addslashes(serialize($prefs));
$prefs = serialize($prefs);

//TODO: Change it back to update
$sql = "REPLACE %ssocial_privacy_preferences SET member_id=%d, preferences='%s'";
Expand Down
2 changes: 1 addition & 1 deletion profile.php
Expand Up @@ -32,7 +32,7 @@
if (count($profile_row) > 0) {
//get privs
$sql = 'SELECT `privileges`, approved FROM %scourse_enrollment WHERE member_id=%d';
$row_en = mysql_query($sql,array(TABLE_PREFIX, $_GET['id']));
$row_en = queryDB($sql,array(TABLE_PREFIX, $_GET['id']));

if ($system_courses[$_SESSION['course_id']]['member_id'] == $_GET['id']) {
$status = _AT('instructor');
Expand Down

5 comments on commit 111a69d

@sudwebdesign
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of subject, but i have attempted to subscribe to atutor.ca (twice 15days samoht & 30min sudwebdesign) for translate ATutor in french and now in mantis but i never received confirmation e-mails with link for confirmation account.

i like ATutor and i translated a lot of texts in my locally version (with retouched for certain JavaScript texts, The apostrophe(') made of Siena as'habit in french).

thanks for your job, its marvelous
Thomas

@atutor
Copy link
Owner

@atutor atutor commented on 111a69d Aug 24, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the email confirmation broke when the new site was released a couple weeks ago. To be fixed as soon as we can get to it. I've manually set you as a translator.

@sudwebdesign
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Greg, but mantis not send my confirmation e-mail (i have tested my reception and it's ok), maybe the mails sender was broken.
Good luck,
Thomas

@atutor
Copy link
Owner

@atutor atutor commented on 111a69d Aug 24, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mantis email confirmation seems to be working properly, just tested. Check your SPAM filters.

@sudwebdesign
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My spam filter is deactivated and i'm looking on my junks in my mailer but it's no here.

i have tested lost password recovery and mantis said:
Password Message Sent
If you supplied the correct username and e-mail address for your account, we will now have sent a confirmation message to that e-mail address. Once the message has been received, follow the instructions provided to change the password on your account.

But not received mail with instruction to reset password.
but the 1and1 webmail are broken now /-#.
Sorry,
But maybe 1&1 hate atutor.ca website, lol.

Thanks for all and for assign the fix to me ;-)
I continue to translate and discover ATutor.
See you soon
Best regard.
Thomas

Please sign in to comment.