Warzone2100 / old-trac-import

Archived Import of (old) Warzone 2100 Trac
0 stars 0 forks source link

new parser for guide #4003

Closed wzdev-ci closed 10 years ago

wzdev-ci commented 11 years ago

resolution_closed type_patch (an actual patch, not a request for one) | by Ezio


still in development.

works on first parsing: cache id, names, functions, research desc, bodies, propolsions

still not working: cache research-function-links, research-weapon-links, structure weapons, research-structure-links, weapon table

research, research prereqs, weapons and structure are parse-able after changing some code


Issue migrated from trac:4003 at 2022-04-16 10:57:01 -0700

wzdev-ci commented 11 years ago

Per changed blocking which not transferred by tractive

wzdev-ci commented 11 years ago

Per changed blockedby which not transferred by tractive

wzdev-ci commented 11 years ago

Per commented


Need help with anything?

wzdev-ci commented 11 years ago

Ezio commented


only cyborg and research that's not generated properly still not sure whether the problem comes from data structure or view :hmm

wzdev-ci commented 11 years ago

Ezio commented


cyborg still not working. maybe typo or something some research not showing too. due to 'Cache research-structure links'??

wzdev-ci commented 11 years ago

Ezio commented


v5 everthing should be generated (properly?) known issue : total time still not working.

wzdev-ci commented 11 years ago

Ezio commented


v6 fixed resultFunctions on case 6 and resultStructures on case 14. research page still not generating some function -- seems fail at wzguide:717 (foreach an empty array). total time still not working--> case 16

wzdev-ci commented 11 years ago

Ezio commented


fixed string message broken or not showing. total time still broken, not sure how does it works :(

wzdev-ci commented 11 years ago

Ezio commented


hopefully 100% change case 8 logic change researchPower -> researchPoints on case 3

wzdev-ci commented 11 years ago

Ezio commented


this guide should be based on campaign data not mp, right?, so should i change 'data/mp/' to 'data/base/'.

wzdev-ci commented 11 years ago

Per commented


The guide should be based on MP. It would be interesting to have a separate guide for campaign, though.

wzdev-ci commented 11 years ago

Ezio commented


here what i have in mind, putting them side by side would be good but need a wide screen, maybe at least 1280. http://i.imgur.com/yfCSW2h.png

http://i.imgur.com/3zHX5zb.png

wzdev-ci commented 11 years ago

Per commented


Could be a good idea, but keep in mind that some structures are not present in both MP or campaign.

wzdev-ci commented 11 years ago

vexed commented


How does one actually run this thing ?

wzdev-ci commented 11 years ago

Ezio commented


Replying to Warzone2100/old-trac-import#4003 (comment:13):

How does one actually run this thing ? i use xampp for it.

  • put guide.wz2100 and data/mp folder inside htdocs
  • then run generate.php from localhost/folder_name/tools/generate.php
wzdev-ci commented 11 years ago

vexed commented


Replying to Warzone2100/old-trac-import#4003 (comment:14):

Replying to Warzone2100/old-trac-import#4003 (comment:13):

How does one actually run this thing ? i use xampp for it.

  • put guide.wz2100 and data/mp folder inside htdocs
  • then run generate.php from localhost/folder_name/tools/generate.php

I do php tools/generate.php and it comes back with: PHP Warning: PHP Startup: apc.shm_size now uses M/G suffixes, please update your ini files in Unknown on line 0 PHP Warning: include_once(tools/persist.lib.php): failed to open stream: No such file or directory ... and so on.

I don't see a 'lib' directory for lib/global.lib.php on line 3. On original site, there is no lib directory either...

Maybe I am just running it wrong ?

wzdev-ci commented 11 years ago

Ezio commented


Replying to Warzone2100/old-trac-import#4003 (comment:16):

PHP Warning: include_once(tools/persist.lib.php): failed to open stream: No such file or directory ... and so on. it's on wz2100.net folder

wzdev-ci commented 11 years ago

vexed commented


The fun continues.

PHP Warning:  PHP Startup: apc.shm_size now uses M/G suffixes, please update your ini files in Unknown on line 0
PHP Warning:  include_once(tools/persist.lib.php): failed to open stream: No such file or directory in ../guide.wz2100.net/htdocs/tools/generate.php on line 6

Obviously, it is there.

ls tools
DiffEngine.php  editguide.php  generate.php  persist.lib.php  template.html  wzguide.lib.php

...

wzdev-ci commented 11 years ago

Ezio commented


is this works? include_once 'persist.lib.php';

wzdev-ci commented 11 years ago

vexed commented


Nope.

PHP Warning:  include_once(../lib/settings.inc.php): failed to open stream: No such file or directory in ../lib/global.lib.php on line 3
PHP Warning:  include_once(): Failed opening '../lib/settings.inc.php' for inclusion (include_path='.:/usr/share/php:/usr/share/pear') in ../lib/global.lib.php on line 3

and on and on...

What should be done is clone, move everything that needs to be move, fix all the paths that assume everything is where it wants it to be, and make everything self-contained. Then include all that in the patch (git add...) and we can finally put it up someplace for comments.

wzdev-ci commented 11 years ago

Ezio commented


for me, i commented out that settings.inc.php on global.lib.php

wzdev-ci commented 11 years ago

Ezio chaeril@... commented


v9 convert function.txt to .ini (rebuild case 5) convert template.txt to .ini add missing stats in weapons

wzdev-ci commented 11 years ago

Ezio chaeril@... uploaded file 0001-new-parser-for-guide-page.2.patch (63.4 KiB)

v9

wzdev-ci commented 11 years ago

vexed commented


Ezio, while you are at it, can you strip out all the comment abilities and anything else that accepts input ?

Right now, that is just commented out.

wzdev-ci commented 11 years ago

Ezio commented


i removed comment, i only see edit as 'else that accept input', maybe it's better if i remove $isadmin and anything related?

wzdev-ci commented 11 years ago

vexed commented


Replying to Warzone2100/old-trac-import#4003 (comment:24):

i removed comment, i only see edit as 'else that accept input', maybe it's better if i remove $isadmin and anything related?

Yeah, feel free to remove all that stuff.

wzdev-ci commented 11 years ago

Ezio commented


v9.5 this should be everything http://db.tt/wFW9Glaz - patch http://db.tt/unQgOCqr - htdocs (image, icon and generated files)

fix case 5 and case 6 not working as intended add function to search in-array-recursive on case 6 to avoid double use name from ini file instead names.txt (template still use names.txt) use longHit for accuracy (3.2) remove $admin, $auth, $reviewer and comments function onsight remove editguide.php update template.html atom and rss link add calculation for upgrade percentage and max upgrade pretty php i forgot other changes teehee~

maybe-need-fix recheck if upgrade calculated properly update jQuery removed comment on case 113 not sure will work as intented or not

wzdev-ci commented 11 years ago

Ezio uploaded file 0001-new-parser-for-guide-page.patch (344.2 KiB)

v9.5

wzdev-ci commented 11 years ago

Ezio uploaded file htdocs.zip (2910.7 KiB)

v6

wzdev-ci commented 11 years ago

Ezio uploaded file 0001-fix-bracket-typo.patch (1.4 KiB)

wzdev-ci commented 11 years ago

vexed commented


That patch don't apply...


:git apply 0001-new-parser-for-guide-page.patch
error: removal patch leaves file contents
error: guide.wz2100.net/htdocs/tools/editguide.php: patch does not apply

What did you base it on ?

wzdev-ci commented 11 years ago

Ezio commented


Replying to [comment:27 vexed]:

That patch don't apply...


:git apply 0001-new-parser-for-guide-page.patch
error: removal patch leaves file contents
error: guide.wz2100.net/htdocs/tools/editguide.php: patch does not apply

What did you base it on ?

that was my fault since i tick remove attachment with the same name every-time, the patch contains all changes made so you need to reset to origin before apply patch. as for typo it's based on that patch.

"C:\Program Files (x86)\Git\bin\git.exe" am --3way --signoff "C:/.../0001-new-parser-for-guide-page.patch"
Applying: new parser for guide page
Done
wzdev-ci commented 11 years ago

vexed commented


Still having the same issues mentioned in [comment:20] & [comment:18] trying to generate it.

It would make more sense to just do what [comment:20] said.

wzdev-ci commented 11 years ago

Ezio commented


i am not sure if you guys planning to rework the site directory, but this is pretty much what i am doing to make generate.php work for development

1. copy whole wz2100.net(git) to xampp/htdocs/wz/
2. that should make wz/guide.wz2100.net and etc.
3. create forums.wz2100.net dir inside wz2100.net dir
4. extract and install phpbb into that folder
5. open wz/wz2100.net/lib/global.lib and comment include_once(dirname(__FILE__) . '/settings.inc.php');
6. run wz/guide.wz2100.net/tools/generate.php
wzdev-ci commented 11 years ago

vexed commented


Replying to [comment:30 Ezio]:

i am not sure if you guys planning to rework the site directory, but this is pretty much what i am doing to make generate.php work for development

1. copy whole wz2100.net(git) to xampp/htdocs/wz/
2. that should make wz/guide.wz2100.net and etc.
3. create forums.wz2100.net dir inside wz2100.net dir
4. extract and install phpbb into that folder
5. open wz/wz2100.net/lib/global.lib and comment include_once(dirname(__FILE__) . '/settings.inc.php');
6. run wz/guide.wz2100.net/tools/generate.php

Basically, none of that directory structure remains, that is why I said to go ahead and add anything that is needed to its own directory. You can also include anything from the phpBB install as well, since this will not have any access to that.

In any case, even following your instructions, it still don't generate anything. It shows no error messages or anything, and nothing seems to have been generated.

wzdev-ci commented 11 years ago

Ezio commented


Replying to [comment:31 vexed]:

It shows no error messages or anything, and nothing seems to have been generated. strange, it should show something like this 'http://i43.tinypic.com/fdgkyd.jpg' when go to case 0 it would create 'cache' dir inside wz dir and case 1-16 generate .inc.php inside cache dir which based on data/mp/

wzdev-ci commented 11 years ago

vexed commented


cache directory is empty.

Odd stuff, as I said, no error message or any message at all when it is run.

wzdev-ci commented 11 years ago

Ezio commented


did you check php_error_log, apache's access.log and error.log.might be mention something useful. i doubt mysql_error records something.

wzdev-ci commented 11 years ago

vexed commented


Replying to [comment:34 Ezio]:

did you check php_error_log, apache's access.log and error.log.might be mention something useful. i doubt mysql_error records something.

Nope, nothing (of value) pops up in the error.log file.

wzdev-ci commented 11 years ago

Ezio commented


might be late, but have you try to run it using new instance of xampp. probably a portable one on a different directory so it wont mess each other settings

wzdev-ci commented 11 years ago

vexed changed _comment0 which not transferred by tractive

wzdev-ci commented 11 years ago

vexed commented


Replying to [comment:36 Ezio]:

might be late, but have you try to run it using new instance of xampp. probably a portable one on a different directory so it wont mess each other settings

Server isn't running xampp at all, it is using PHP Version 5.3.10-1

Is xampp a requirement ?

wzdev-ci commented 11 years ago

Ezio commented


Replying to [comment:37 vexed]:

Is xampp a requirement ?

No, i use php 5.5.1http://pastebin.com/T4sSbtX2 while developing it. i just test it on 5.3.1http://pastebin.com/KveC9Vj4 (first i though there is new function that i used which need 5.5) but i didn't see any error and guide were generated.

wzdev-ci commented 11 years ago

vexed commented


Don't know what to say, it just won't work, and no errors in any logs that I have looked at.

I still think all the directories and whatever else it looks for should be rearranged, as talked about in comment:31.

That means, main directory should hold all files it needs to generate the data, with the exception of the data/ directory for the stats themselves.

wzdev-ci commented 10 years ago

vexed changed status from new to closed

wzdev-ci commented 10 years ago

vexed changed resolution from ` toclosed`

wzdev-ci commented 10 years ago

vexed commented


Since http://betaguide.wz2100.net is now up, I think we can close this ticket.

Sorry we couldn't get it working. :(