| Author |
 Topic  |
|
laser
Advanced Member
Australia
3859 Posts |
 Posted - 08 August 2003 : 19:51:09
|
Check!
Next step :
4) Between the While and Wend lines, create a new line and add this :
Response.Write DisplayMedalIcon(mymedals("member_id"))
mymedals.movenext
It should now be close to working. |
 |
|
|
Alfred
Senior Member
USA
1527 Posts |
Posted - 08 August 2003 : 20:00:10
|
Something's orking - at least we're getting error messages again: quote:
Error Type:
Microsoft JET Database Engine (0x80040E14)
Syntax error (missing operator) in query expression 'Member_ID ='.
/BG/forums/decorated3.asp, line 6
|
Alfred
The Battle Group
CREDO
|
|
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 08 August 2003 : 20:12:57
|
Change this line :
strSql = "SELECT * from member_medals WHERE Member_ID = " & Request("id")
to this :
strSql = "SELECT * from member_medals WHERE Member_ID = " & m_id
This is really the code from your inc function file ?? ... I didn't think it would work like that, but as long as it does you're fine. |
|
|
|
Alfred
Senior Member
USA
1527 Posts |
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 08 August 2003 : 20:53:39
|
arrghhhh, I was hoping we wouldn't get this error but the fix is simple.
On these lines ONLY:
82, 84, 86,88, 89, 93, 94
change "mymedals" to "member_medals"
It looks like you altered the function you pasted in before, but it looks ok now. |
|
|
|
Alfred
Senior Member
USA
1527 Posts |
Posted - 08 August 2003 : 22:07:28
|
| Wow, that shows a string of medals now, but no names with them! |
Alfred
The Battle Group
CREDO
|
|
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 08 August 2003 : 22:21:48
|
PERFECT  now change :
Response.Write DisplayMedalIcon(member_medals("member_id"))
to this :
Response.Write member_medals("m_name") & " " & DisplayMedalIcon(member_medals("member_id")) & "<br>"
|
|
|
|
Alfred
Senior Member
USA
1527 Posts |
Posted - 08 August 2003 : 22:44:34
|
| We must be close, but it still lists the names more than once! |
Alfred
The Battle Group
CREDO
|
|
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 08 August 2003 : 23:06:42
|
5) in the SQL statement that was originally there, change this line :
strSql = "SELECT M.MEMBER_ID, M.MEDAL_ID, M1.M_NAME " & _
to this
strSql = "SELECT DISTINCT M.MEMBER_ID, M1.M_NAME " & _ |
|
|
|
Alfred
Senior Member
USA
1527 Posts |
Posted - 08 August 2003 : 23:47:30
|
You did mean on line 79, right?
quote:
Error Type:
Microsoft VBScript compilation (0x800A03EA)
Syntax error
/BG/forums/decorated3.asp, line 80
'strSql = "SELECT M.MEMBER_ID, M.MEDAL_ID, M1.M_NAME " & _
|
Alfred
The Battle Group
CREDO
|
Edited by - Alfred on 08 August 2003 23:52:27 |
|
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 09 August 2003 : 00:04:30
|
| You can't comment the line following a line continuation, so delete the commented line, or swap the order |
|
|
|
Alfred
Senior Member
USA
1527 Posts |
Posted - 09 August 2003 : 11:16:11
|
Oh, I see.
So much to learn - so little time! 
Well, it looks like you did it!
We got each name listed once, and all medals that belong to it! |
Alfred
The Battle Group
CREDO
|
|
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 09 August 2003 : 17:49:29
|
| There are a few little things to clean up, but no biggies and nothing that will affect the page at all. |
|
|
|
Alfred
Senior Member
USA
1527 Posts |
Posted - 09 August 2003 : 19:47:23
|
You mean code-cosmetically, right?
Well, nobody expects clean code on my site I guess.
I think now that you have laid out the stucture I can play around with the size and formatting a bit before I get stuck again.
Thanks a heap for now, laser! |
Alfred
The Battle Group
CREDO
|
|
|
|
laser
Advanced Member
Australia
3859 Posts |
Posted - 09 August 2003 : 20:02:25
|
| As I said, little things, and everything works so you may not want to play with it at all, but the maintenance when you add medals is not the best. I don't mean medal-member links, I mean making a new medal available. |
|
|
|
Topic |
|
Topic Locked
