API(Alternate branch): Added first few calls without using get_courses_m... #25

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
@sdaityari
Owner

sdaityari commented Jul 4, 2014

Creating the API without use of functions get_*_main

- "one_row" => $one_row,
- "member_id" => $member_id
- ));
+ return;

This comment has been minimized.

@sdaityari

sdaityari Jul 5, 2014

Owner

I just commented out everything so that there is no error anywhere in the API.

@sdaityari

sdaityari Jul 5, 2014

Owner

I just commented out everything so that there is no error anywhere in the API.

+ "INNER JOIN %scourse_enrollment ce ON c.course_id = ce.course_id ".
+ "WHERE ce.member_id = %d";
+
+ if ($clause) {

This comment has been minimized.

@anvk

anvk Jul 7, 2014

is it possible to add the "AND" piece to the clause inside create_SQL_clause() ?

@anvk

anvk Jul 7, 2014

is it possible to add the "AND" piece to the clause inside create_SQL_clause() ?

This comment has been minimized.

@sdaityari

sdaityari Jul 7, 2014

Owner

Adding "AND" or "WHERE" within the clause function is possible, if I supply another argument. Will look into it.

@sdaityari

sdaityari Jul 7, 2014

Owner

Adding "AND" or "WHERE" within the clause function is possible, if I supply another argument. Will look into it.

This comment has been minimized.

@anvk

anvk Jul 7, 2014

See if there is an elegant way to do this. If not your code looks good to me as it is.

@anvk

anvk Jul 7, 2014

See if there is an elegant way to do this. If not your code looks good to me as it is.

@sdaityari sdaityari closed this Jul 8, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment