21
timgno
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/5 10:00

  • timgno

  • Module Developer

  • Posts: 1504

  • Since: 2007/6/21


Point 3) I had seen in my tests.

Had already provided for this with the addition of the field field_numb, in the table tdmcreate_fields.

In fact, this field is used only for this work, otherwise I would not have added as unnecessary since there is already a field table_nbfiles in the table tdmcreate_tables.

The work that must be done is to add loops to control this field_numb which table belongs to and it must be done in the file admin/tables.php, case save.

Point 6) Sorry! This is only because I was doing some tests with the var_dump function of php. It is not a bug will be deleted at the appropriate time.

Point 10) At the moment it might be a simple bug, but I assure you it works.

You just have to choose a radiobutton in tab fileds when new or edit that you feel may be the main field

To make you understand I'll show you what it is for!

If you go in the category file or other files, you will notice that there is a line of php class Criteria with such $criteria->setSort('cat_id ASC, cat_name');

The cat_name is main field and you will find it also in the class file.

I hope this has cleared up a lot of ideas

22
goffy
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/5 10:19

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


hi timgno

Quote:
Point 6) Sorry! This is only because I was doing some tests with the var_dump function of php. It is not a bug will be deleted at the appropriate time.


The problem is not the output from var_dump.
the problem is, that in table tdmcreate_tables in column "table_mid" and in table tdmcreate_fields in column "field_mid" you find always 257, not the id of the module

23
timgno
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/5 10:22

  • timgno

  • Module Developer

  • Posts: 1504

  • Since: 2007/6/21


Are you sure you have not created so many modules up to 257?

24
goffy
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/5 14:57

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


Hi timgno

yes, fresh installed.

but please see pic http://wedega.com/downloads/tdmcreate01.png

sometimes there is 257 in field "table_mid", sometimes the correct id
I have no explanation till now, but I try to find out

25
timgno
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/5 18:39

  • timgno

  • Module Developer

  • Posts: 1504

  • Since: 2007/6/21


You have to completely uninstall the module, otherwise those fields do not disappear.

Also uninstall your new modules created

26
goffy
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/7 6:17

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


hi timgno

I found the problem

in class/tables.php line 109 there is
Quote:
$mods_select->addOption($action, _AM_TDMCREATE_TABLE_MODSELOPT);

if you forget to select a module (as it was in some cases of my tests) you get back the action string as table_mid. In my case action string was
Quote:
/xoops257/htdocs/modules/TDMCreate/admin/tables.php?op=new
so I got back 257

please change line 109 to Quote:
$mods_select->addOption(0, _AM_TDMCREATE_TABLE_MODSELOPT);

and add a check like this
if( ($tablemid == 0) {
    
redirect_header('tables.php?op=new'10_AM_TDMCREATE_ERROR_NO_MODULE_SEL);
    exit();
}
to admin/tables.php before saving operation

I'll continuing tests


27
timgno
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/7 6:59

  • timgno

  • Module Developer

  • Posts: 1504

  • Since: 2007/6/21


@goffy

Poured code since the control of select box modules in tab admin/tables is done by javascript instead of using 0, we use two single quotes as string in class/tables.

28
goffy
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/7 7:08

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


Hi timgno, I don't understand what you mean

29
timgno
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/7 7:12

  • timgno

  • Module Developer

  • Posts: 1504

  • Since: 2007/6/21


Download it from svn again and see the changes that I just made,

Compare them with the previous commit

Go to the tab Tables and without entering any data, click on Submit

30
goffy
Re: TDMCreate 1.91 alpha 1 for Testing
  • 2014/8/7 7:37

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


works

I'll continuing

Login

Who's Online

397 user(s) are online (294 user(s) are browsing Support Forums)


Members: 0


Guests: 397


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Nov 30
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits