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

Fixing boolean check for stats (Issue #1270) #1273

Merged
merged 1 commit into from Nov 4, 2013
Merged

Fixing boolean check for stats (Issue #1270) #1273

merged 1 commit into from Nov 4, 2013

Conversation

ditorelo
Copy link
Contributor

eaf1a69 half fixed the issue with connection problems.

The check for $xml == false is always true, and so the stat_id and stat_key returned by the tracker are never saved on the database. This never lets new deployments get their stats started.

eaf1a69 half fixed the issue with connection problems.

The check for $xml == false is always true, and so the stat_id and
stat_key returned by the tracker are never saved on the database. This
never lets new deployments get their stats started.
@ditorelo ditorelo mentioned this pull request Oct 29, 2013
@hniamkey
Copy link

@ditorelo thank for your help but i don't understand how to fix this bug, what is the file where found "xml==false", can you explain please. Thank.

@ditorelo
Copy link
Contributor Author

Hi @hniamkey,

Maybe have a look at ditorelo@9e8dd03 - you have to update the file application/models/stats.php with the changes shown there.

@hniamkey
Copy link

@diletero hop excuse me i don't make attention to the link below. Awesome bug fix. Thank.

rjmackay added a commit that referenced this pull request Nov 4, 2013
Fixing boolean check for stats (Issue #1270)
@rjmackay rjmackay merged commit 1f4734b into ushahidi:develop Nov 4, 2013
@ditorelo ditorelo deleted the 1270 branch November 4, 2013 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants